Closed
Bug 345215
Opened 18 years ago
Closed 16 years ago
no tooltip for bookmarks overflow chevron button
Categories
(Firefox :: Bookmarks & History, defect)
Firefox
Bookmarks & History
Tracking
()
VERIFIED
FIXED
Firefox 3.6a1
People
(Reporter: moco, Assigned: awuest)
Details
(Keywords: late-l10n, polish, verified1.9.1)
Attachments
(1 file, 1 obsolete file)
1.96 KB,
patch
|
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
no tooltip for bookmarks overflow chevron beltzner, do you think we need one? I'm using 2.0 b1 official bits on winxp.
Reporter | ||
Comment 1•18 years ago
|
||
fwiw, safari has "Show more bookmarks."
Reporter | ||
Comment 2•18 years ago
|
||
beltzner suggests: "Show more bookmarks" (note safari does "Show more bookmarks.")
Assignee | ||
Comment 3•16 years ago
|
||
Shows a "Show more bookmarks" tooltip when hovering over the chevron. Makes sure that menuitems contained in the chevron's menu don't inherit the chevron's tooltiptext.
Attachment #308258 -
Flags: review?(seth)
Comment 4•16 years ago
|
||
Comment on attachment 308258 [details] [diff] [review] v1 forwarding review to gavin.
Attachment #308258 -
Flags: review?(seth) → review?(gavin.sharp)
Updated•16 years ago
|
Component: Bookmarks → Places
OS: Windows XP → All
QA Contact: bookmarks → places
Hardware: PC → All
Comment 5•16 years ago
|
||
Comment on attachment 308258 [details] [diff] [review] v1 This looks fine for me, though we're past string freeze so we're unlikely to be able to take this for Firefox 3.
Attachment #308258 -
Flags: review?(gavin.sharp) → review?(mano)
Updated•16 years ago
|
Assignee: nobody → awuest
Summary: no tooltip for bookmarks overflow chevron → no tooltip for bookmarks overflow chevron button
Assignee | ||
Comment 6•16 years ago
|
||
Should this bug have [polish-interactive] and [polish-easy] set?
Comment 7•16 years ago
|
||
Comment on attachment 308258 [details] [diff] [review] v1 r=mano
Attachment #308258 -
Flags: review?(mano) → review+
Comment 8•16 years ago
|
||
The patch for bug 237592 made the tooltiptext="" part unnecessary.
Updated•16 years ago
|
Keywords: checkin-needed
Updated•16 years ago
|
Attachment #308258 -
Attachment is obsolete: true
Comment 9•16 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/b94206f56946
Status: NEW → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.2a1
Comment 10•16 years ago
|
||
Comment on attachment 352897 [details] [diff] [review] updated patch asking approval for 1.9.1, no risk.
Attachment #352897 -
Flags: approval1.9.1?
Comment 11•16 years ago
|
||
Comment on attachment 352897 [details] [diff] [review] updated patch a191=beltzner; no risk, but text change which needs to land before Dec 31 to meet string freeze
Attachment #352897 -
Flags: approval1.9.1? → approval1.9.1+
Updated•16 years ago
|
Comment 12•16 years ago
|
||
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/a9233111a848
Keywords: fixed1.9.1
Comment 13•15 years ago
|
||
Verified Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.1b3pre) Gecko/20090130 Shiretoko/3.1b3pre
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
Comment 14•15 years ago
|
||
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•