Closed
Bug 1393799
Opened 7 years ago
Closed 7 years ago
[Form Autofill] The red border of the invalid email box is incomplete
Categories
(Toolkit :: Form Manager, defect)
Tracking
()
VERIFIED
FIXED
mozilla57
People
(Reporter: zstimi, Assigned: scottwu)
Details
(Whiteboard: [form autofill])
Attachments
(3 files)
61.52 KB,
image/png
|
Details | |
59 bytes,
text/x-review-board-request
|
lchang
:
review+
|
Details |
756 bytes,
patch
|
lchang
:
review+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
[Environment:] Windows 8.1 x64, Mac Osx 10.11 56.0b6 20170825011442 [Steps:] Preconditions Go to Preferences/ Privacy and Security / Form Autofill / Enable Profile autofill. (default in Nightly) Make sure you have at least one saved address. 1. Open Firefox 2. Add addresses, in Email box put an invalid email 3. Save the address 4. Edit the address with invalid email [Actual Result:] The red border of the email box is incomplete on the right side. [Expected Result:] The border of the email box is red all around.
Updated•7 years ago
|
Whiteboard: [form autofill]
Comment hidden (mozreview-request) |
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8902547 [details] Bug 1393799 - Add extra space around forms to ensure invalid fields are displayed properly. https://reviewboard.mozilla.org/r/174148/#review179400
Attachment #8902547 -
Flags: review?(lchang) → review+
Pushed by lchang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7adf0301c546 Add extra space around forms to ensure invalid fields are displayed properly. r=lchang
Assignee | ||
Comment 5•7 years ago
|
||
Rebased for beta uplift: since editDialog.css does not exist in beta 46, the padding rule is added to editAddress.css instead.
Attachment #8902613 -
Flags: review?(lchang)
Updated•7 years ago
|
Attachment #8902613 -
Flags: review?(lchang) → review+
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7adf0301c546
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Assignee | ||
Comment 7•7 years ago
|
||
Comment on attachment 8902613 [details] [diff] [review] Bug 1393799 - Patch for beta 46 Approval Request Comment [Feature/Bug causing the regression]: Not a regression. Bug exists since 1019471, when the feature first landed. [User impact if declined]: Users would notice the border on the right is missing when entering an invalid email address. [Is this code covered by automated tests?]: No. Page layout is not covered in tests. [Has the fix been verified in Nightly?]: Yes I can verify that it's been fixed on Nightly. [Needs manual test from QE? If yes, steps to reproduce]: No. [List of other uplifts needed for the feature/fix]: None. [Is the change risky?]: No. [Why is the change risky/not risky?]: The style change is minor and only affects editAddress.xhtml [String changes made/needed]: None.
Attachment #8902613 -
Flags: approval-mozilla-beta?
Comment 8•7 years ago
|
||
Comment on attachment 8902613 [details] [diff] [review] Bug 1393799 - Patch for beta 46 Minor fix for auto formfill, should be fine to uplift for beta 8.
Attachment #8902613 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 9•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/76813e4100ea
Comment 10•7 years ago
|
||
Verified as fixed under Windows 10x64 and macOS 10.12 using 56.0 20170918210324 and 57.0a1.
Status: RESOLVED → VERIFIED
Updated•7 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•