Comments by Brian A Stephens (Top 75 by date)

Could you post more code? Post the relevant HTML and HTTPGET request. Thanks.

So all you are really doing is declaring that you need a regular expression. Not a very specific statement, and certainly not a question.

Reason for my vote of 5 \n good explanation

Can you show us the relevant HTML? What error/result did you get?

Reason for my vote of 4 \n this could be very useful. thanks.

Reason for my vote of 5 \n sleek styling!

Reason for my vote of 5 \n very good

yes, or .css('max-height', '100%');

It looks like Torakami was trying to use the deprecated toggle method (removed in jQuery 1.9) that allowed for alternating event handlers on an element (http://api.jquery.com/toggle-event/). So in this case, he wasn't trying to hide and show the elements.

This also corrects the issue of allowing underscores, which presumably he did not want. Now, if he truly wants to allow "spaces" rather than any whitespace, that's another change.

Reason for my vote of 5 \n good to know

You don't have any #ad_whatever DIVs in your HTML. Where are they?

I would also caution against your use of $(this).attr('class'), since it will return ALL classes of the element. You may very well add another class for styling purposes that will remain in both the "like" and "unlike" states. It's more precise to use $(this).hasClass('like').

Your handler will be triggered by ANY element with the class "chk-jobOffer" being clicked. Then your conditional will check if the FIRST such element is checked, which is not necessarily what you want. You are assuming there's only one matching element.

I realized that I misunderstood what the navigation was controlling. Try my update for a complete solution.

I see that you may have two possible problems. You need to define the regex using a verbatim string literal, so the backslashes aren't interpreted in the string itself: string pattern = @"(\$\d+(,\d{3})*|\d+(,\d{3})*\$)". Secondly, if you are using Regex.Matches, it should return ALL matches, not just the first.

I added a jsFiddle to show the difference with the new CSS. Try it out.

The code tells it all! You are trying to validate each character on its own, but your requirements make this impossible ("dot should come in the middle of the string and only once"). Because of the dot validation, you must consider the entire string, not just one letter. You will need to change this line:
var str = $('#txtFileName').val();

But you still have the problem that keypress happens BEFORE the new text becomes part of the input field value. You need to validate the text that includes the new key. That can only happen in keyup, but that's too late to prevent the key's action.

You need to either validate the entire filename on submit rather than entry, or keep track of a "previous" value for the input so you can revert it back in a keyup handler if the new value doesn't validate.

OK, so my regex is what you need. It allows for the possibility of one dot followed by more letters or numbers. I'm not sure why you're not getting it to work. Can you post your code? If you are using
new RegExp(" ... ")
syntax, be sure you escape the escape character (double backslash) because it's also an escape character within quotes.

Reason for my vote of 4 \n good, practical code

Agree completely with SA. And what are those <left> tags?

Reason for my vote of 5 \n useful information

Reason for my vote of 5 \n Great idea. And this is all front-end optimization, based on what you can see as a user. Just think of the room for improvement having access to all the code.

Wow, this is unclear. It would help to see what "some code" means. And what do you mean by "try to remove the code for the watermark"?

Realize, though, that "\w" matches all alphanumerics and underscore. If he really means "alpha", you have to use "[A-Za-z]".

Also, the expression allows "100.%", which is strange. The "0{0,2}" should be changed to "0[1,2}" to force at least one digit after the decimal.

Reason for my vote of 5 \n good explanation.

Note: you need to clean up some errant text in the line with "if Name field length >=5 then flag ...". You must have pasted it in by mistake.

Reason for my vote of 5 \n good advice, and written clearly

You tagged this question with HTML5 only, not Javascript. With HTML5 only, the closest you'll get is the new pattern attribute of input tags. It will prevent any characters that don't match the specified regex.

But if you want to allow the copy/paste, but clean it for the user, you'll need Javascript or server-side code. Write a JS function that utilizes regex to replace the invalid characters, bound to the onkeypress event.

Reason for my vote of 5 \n very useful

Or simplify the function to one line:
$("#spnChk").toggleClass("disabled checked");

You just have to add the "disabled" class to spnChk so it defaults correctly.

Now that makes sense!

Reason for my vote of 5 \n handy tip

This regex allows any number of digits. He said it should be 10 max. There should obviously be a minimum too, but that was not stated.

This has nothing to do with jQuery

After I posted my answer and saw yours, I realized that he must be looking for an element with value="art6" INSIDE an anchor because "value" is not a valid attribute of an anchor tag.

Be sure you specify the RegexOptions.Multiline option to change the meaning of ^ and $ to be per line.

What result are you getting? The caret ^ ensures that the line has to start with the matching text, so it should match only one line from your sample.

Your regex allows numbers 11-19 also, which shouldn't be included.

Reason for my vote of 5 \n great tips! this all makes sense.

Reason for my vote of 3 \n good tip, but all it did was document the existing jQueryUI call

Reason for my vote of 5 \n good tip

Reason for my vote of 5 \n short and to the point

To match a dash (hyphen) in a character class, the dash needs to be either first or last, or be escaped. The syntax you have given may work inadvertently, but is not correct because of the space at the end.

Reason for my vote of 5 \n good intro

Reason for my vote of 3 \n Good try, but it doesn't work well. I tried 2013-05-31 (with today's date being 2013-06-28), and the output was 1 month and -2 days. While that's true in a very awkward way, it's not the intended output. Also, why do you initialize the "Days" and "Months" variables but never use the initial values?

I think this is a valuable algorithm to work out, so thank you for the effort and hopefully you can work out these bugs.

Reason for my vote of 5 \n clean look, achieved with small footprint

Please also label your question correctly. "Browser compatibility issue" is very vague and there are at least 4 extra tags that aren't relevant (this has nothing to do with .NET or C#).

It's difficult to determine exactly how this code will be used. Can you provide the accompanying HTML, or even better, a jsFiddle example? Also, it looks like you have a mistake in the CSS: there are two definitions for ".tip". Did you mean the second one to be ".tipTop"?

So my solution above should do what you want. If so, please "accept" it as the answer; if not, then clarify what you want different. Thanks.

Reason for my vote of 5 \n good trick

It's very strange that all you are validating about a phone number is its length. Why are you not using a regex to ensure that it's all numbers, dashes, etc., of the correct format?

You should probably provide a jsFiddle. What you have posted here includes a 1px font size on all the menu links, so the menu is not readable. It's also unclear how you WANT the menu to look when there isn't enough room for all the items on one line. There are many ways to style a menu for such circumstances.

What does this have to do with asp.net or an IFrame?

You've gotta provide more information than this! Your code implies that it's more than just a click event that's broken, since you have an alert that presumably isn't firing either (or does the first alert work?). Check the generated HTML page to ensure that the reference to jQuery is correct. Tell us if there's any error from the console, etc.

I was wondering the same thing. Got my downvote.

I tried this code in IE, and it works. Can you post the code to jsFiddle, as it was rendered by MVC? I didn't use MVC; I entered the <label> and <input> tags manually. Also, what version of IE were you trying?

Please post any JS related to the above code and reveal the html SOURCE, rather than rendered. It looks like you have a clickimagechange() function that we need to see. From piecing apart the HTML in the page source of your post, it looks like your LI and UL tags are all scrambled. I would rather see just the relevant code here than go out to the full external site. Thanks.

[In reply to mvamsianil] Yes, she is using jQuery (notice the jQuery selectors in the code). jQuery code should generally be inside a document.ready callback (I'm not sure whether this is), but I don't understand your comment. The code that DEFINES the function will be run only once whether it's in a document.ready or not - it's just the timing of the definition that would change. The click event should call the function every time.

Have you debugged in the browser to step through and see what's happening?

The $ in the selector indicates that the id attribute should END with the given value, rather than being equal. It's one approach when using ASP.NET web forms, which generate their own IDs on the fly.

Please clarify whether you are using the jQuery UI tab control - that's my guess. It's hard to determine what's wrong with your code without seeing it. Can you create a jsFiddle?

The proper syntax for a relative change is += or -= not just the "+25%" you have above. The "+25%" would probably be interpreted by most browsers the same as "25%" - just an indication that it's a positive number.

This code returns the height of ONE item (the largest item) within the div, not the cumulative total height. The question is about the total height.

It's unclear what you are trying to do with this, especially because your code isn't valid. What are the "removed" CSS styles supposed to be? I changed most of them to "position" and got an effect where each time you click, div1 slides further to the left and div2 slides further to the right. What else do you want? What's the point of checking whether the container is visible, since it will always be?

Shouldn't that be "border-left-color" instead of "borderLeftColor" in the css() call?

You are right, sorry. I changed my vote. Your answer is valid and applies to some alternate situations too. When I first read your answer, I misunderstood what you were saying.

I added working code to the solution, with a jsFiddle example.

When you inspect the element after the page loads, do the dynamic radio buttons look right? What if you copy the generated HTML and paste it into the document, temporarily bypassing the dynamic generation: does it work? You should just rule out subtle syntax differences before assuming it's a problem with being dynamically generated. For instance, your dynamic tags have the XHTML-style end slash, but the static ones don't.

So it looks like you are using the jQuery Validation Plugin. But what is your question? Have you tried this code and are getting an error?

Ah, yes: negative lookahead; that's the elegant way to do it. However, the regex you provided doesn't match lines that fail the requirement of a pipe in the 11th position. With a slight modification, it will match those too: ^(?!\d{10}\|).*$