Please report any other irregularities here.
From Bugzilla Helper: User-Agent: Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:0.9.3+) Gecko/20010821 Netscape6/6.1b1 BuildID: 20010821 Link Properties: The Link Properties context panel contains instructions that cannot be followed. The statement under the Link Location field says ``(Clear text to remove existing link form page.),'' however clearing the text does not allow removal of existing link. The statement under the Link Location field says ``(Clear text to remove existing link form page.)'' However if you do so you have to replace the removed information with something in order to make the [OK] button available once again. When the Link Location field is blank the [OK] button is greyed-out. The parenthetical instruction implies that one should be able to ``un-linkify'' a piece of text on a page by clearing the field and clicking [OK]. Reproducible: Always Steps to Reproduce: (1) Select the link and click Link toolbar icon, Link Selection from the menu command, or Link Selection from the keyboard command (2) The target web page location appears in the Link Location field (3) Following the instructions to clear the text from the field Actual Results: The [OK] button is no longer available; it becomes greyed-out. Expected Results: Clicking [OK] should return user to the document being edited and the former link is returned to mere plain text. This may merely be a cosmetic flaw, instead of a failure in functionality.
Assignee: brade → cmanske
Hardware: PC → All
Good catch. This regression was caused by the recent fix for bug 90230.
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Target Milestone: --- → mozilla0.9.4
note that this fix removes the line: SetElementEnabledById("MakeRelativeUrl", enable); that was accidentally checked in with the fix to 90230. It has no effect, but should be removed (it is part of the fix for bug 72583, which isn't finished yet.)
Keywords: patch, review
Whiteboard: FIX IN HAND need r=, sr=
firstname.lastname@example.org with parens around the > check.
Whiteboard: FIX IN HAND need r=, sr= → FIX IN HAND need r=
As per kin's suggestion, the changed line will be: var enable = insertNew ? (dialog.hrefInput.value.trimString().length > 0) : true;
Whiteboard: FIX IN HAND need r= → FIX IN HAND
a=asa on behalf of drivers.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Keywords: patch, regression, review
Resolution: --- → FIXED
Whiteboard: FIX IN HAND
Verified on 8-30 build
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.