Closed Bug 1174122 Opened 5 years ago Closed 5 years ago

Update the URL bar autocomplete popup's star icon so it is visible within selected items

Categories

(Firefox :: Theme, defect, P1)

Unspecified
Windows
defect

Tracking

()

VERIFIED FIXED
Firefox 42
Tracking Status
firefox41 --- affected
firefox42 --- verified

People

(Reporter: dao, Assigned: dao)

References

(Blocks 2 open bugs)

Details

Attachments

(3 files, 3 obsolete files)

from bug 1173746 comment 4:

> [...] you basically can't
> see the (dark blue) star for bookmarked autocomplete items with the default
> windows highlight color that we now use? I suppose this is why it used to be
> yellow [...] it was
> obviously already an issue for winxp luna, classic, and some high contrast
> themes (though we don't seem to have picked up on it) but we might still
> want to fix it in our win10 polish project now that it applies to win8 and
> win10 as well,
needinfo for the image
Flags: needinfo?(shorlander)
Flags: qe-verify+
Can we just use the same image as on OS X?
IIRC it's a completely white star as well. Not sure if the outline/proportions are any different, but I'd be surprised if they were...
Looking at the existing bookmark-XP.png and bookmark.png... Both have an embossed look, whereas the new one doesn't. Is this a deliberate difference or can we get rid of one or both of the older variants (while still making sure we use the white star for selected items depending on the OS, obviously)?
Flags: needinfo?(shorlander)
(In reply to Dão Gottwald [:dao] from comment #5)
> Looking at the existing bookmark-XP.png and bookmark.png... Both have an
> embossed look, whereas the new one doesn't. Is this a deliberate difference
> or can we get rid of one or both of the older variants (while still making
> sure we use the white star for selected items depending on the OS,
> obviously)?

The difference is deliberate. I am not actually sure if we are using the first image of the sprite anywhere. The same icon lives in Toolbar.png. If it isn't used we could remove it entirely and just have the blue and inverted in this sprite.
Flags: needinfo?(shorlander)
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attached patch patch (obsolete) — Splinter Review
Attachment #8631629 - Attachment is obsolete: true
Attached patch patch (obsolete) — Splinter Review
another typo *sigh*
Attachment #8631633 - Attachment is obsolete: true
Attached patch patchSplinter Review
Attachment #8631657 - Attachment is obsolete: true
Attachment #8631746 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8631746 [details] [diff] [review]
patch

Review of attachment 8631746 [details] [diff] [review]:
-----------------------------------------------------------------

I take it we don't mind winxp hidpi getting the win8/win10 star?

Also, the bookmarks button on the bookmarks toolbar on win8 needs margin/padding love. :-(
Attachment #8631746 - Flags: review?(gijskruitbosch+bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/56ae687cc98a
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Reproduced the bug with Aurora 41.0a2 (2015-07-28).

This is verified fixed on Nightly 42.0a1 (2015-07-28), using Windows 10 Pro x64 (Build 10240).
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.