using 2/7 build of netscape 1) launch netscape 2) launch composer 3) insert text ("hello world") 4) hit carriage return 5) insert named anchor "foo" 6) highlight "hello world" 7) bring up link props click on "foo" in named anchor list it doesn't select the first time. you have to click on "foo" again to select it. all platforms.
Sujay and I both saw this one, assigning to cmanske
Simple fix is ready.
still not fixed...just tried it on 2/26 build.
This is really a different bug than original, so I change the summary. Clicking on a named anchor takes the name and appends it to "#" in the Location (href) input field, but unselects the named anchor in the list, which is confusing to user. Easy to fix.
Some questions/comments on the 02/28/01 14:00 patch: 1. Avoid double negatives, they can sometimes be confusing: if(!dontChange) How 'bout changing the var name to gClearListSelections, reversing the init value and checking: var gClearListSelections = true; ... if (gClearListSelections) 2. Is there a way to avoid using a global? Like making it so you can do something like dialog.clearListSelections? Not a requirement, but I hate globals. 3. Fix the indentation of the Unselect calls you are pushing under your new if statement. 4. Remove the ChangeText() call that you commented out.
There's no advantage to not using "global", which are equivalent to window.gVariable anyway, and scope to just the dialog's window. Ok about other comments.
verified in 4/19 build. however there is a new problem in that OK button is not highlighted after clicking on named anchor in list. see Bug 76680