Add tooltip message for refresh button in Storage Inspector section
Categories
(DevTools :: Storage Inspector, enhancement, P2)
Tracking
(firefox-esr60 wontfix, firefox63 wontfix, firefox64 wontfix, firefox65 wontfix, firefox66 wontfix, firefox67 verified)
People
(Reporter: cfogel, Assigned: rfgo, Mentored)
Details
(Keywords: good-first-bug)
Attachments
(2 files, 1 obsolete file)
[Affected versions]: - 60.3.0esr , 63.0.1, 64.0b6, 65.0a1 (2018-11-04) [Affected platforms]: - Win10, macOS10.13, [Steps to reproduce]: 1. Launch Firefox, access any webpage; 2. Open devTools - Storage Inspector tab; 3. Hover over the Refresh button in the inspector section; [Expected result]: - tooltip is displayed; [Actual result]: - no tooltip displayed; [Regression range]: - not a regression, the refresh button seems to not have a tooltip added from the start; - pre-implementation: 2017-08-30; - post-implementation: 2017-08-31;
Comment hidden (obsolete) |
Assignee | ||
Comment 2•4 years ago
|
||
Hi Cristi and Mike,
Since the bug was still on here, I went ahead and tried to tackle it. I've attached a screenshot below and wanted to request a code review.
Thanks.
@Rainier That looks great... can you attach the patch?
Assignee | ||
Comment 5•4 years ago
|
||
Assignee | ||
Comment 6•4 years ago
|
||
@rainier r+... would you like me to land your patch for you?
Assignee | ||
Comment 9•4 years ago
|
||
Since this is my first bug, just wanted to double check, in the future the mentors will be landing the patches right?
Updated•4 years ago
|
Comment 10•4 years ago
|
||
Pushed by mratcliffe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9b783af3efdd Added tooltip to refresh button of storage inspector section. r=miker
I have queued it for landing, thanks for working on it!
(In reply to Rainier Go [:rfgo] from comment #9)
Since this is my first bug, just wanted to double check, in the future the mentors will be landing the patches right?
If you land lots of patches you can get permission to land things but for the moment it will be the mentors. You are best to ask them to land it for you because they may not know whether you have permission to land things yourself.
You can find more good first bugs here:
https://bugs.firefox-dev.tools/?easy&tool=all
Comment 12•4 years ago
|
||
bugherder |
Updated•4 years ago
|
Updated•4 years ago
|
Comment 13•4 years ago
|
||
Confirmed the issue with Firefox 65.0a1 (2018-11-05) on Windows 10.
Fix verified with Firefox 67.0a1 (20190227163104), Windows 10 x64 and macOS 10.13.6.
Description
•