| CARVIEW |
Select Language
HTTP/2 200
accept-ranges: bytes
content-type: text/html; charset=UTF-8
set-cookie: Bugzilla_login_request_cookie=e7vY2PoYnO4t; path=/; secure; HttpOnly; SameSite=Lax
set-cookie: github_token=C9j0u9uXGGq1EhsPS3dR2eRBuujj4hpe2LhvHbubJwNuGg157LdyP3EpGSmzlpHCWGjmbJOzzVlu14Ljp94Ezw3o9VYYbzEjn3A7XO5ZvrilbwowaJTUX0T14Hcs2lPiYcoihC50txGYR31h8l7V3UKHMXxxImQG0gIT9WMqVOf5QU7YN970D9PIqAXyh6pX9uDR4oRYCsbfNoGcIbpygRZUfyBkXTKceHDijQZqYnCW0rdNCAA5VfHTSaast9o6; path=/; secure; HttpOnly; SameSite=Lax
via: 1.1 google, 1.1 varnish, 1.1 varnish
x-content-type-options: nosniff
x-xss-protection: 1; mode=block
strict-transport-security: max-age=31536000; includeSubDomains
content-encoding: gzip
referrer-policy: same-origin
server: nginx
x-frame-options: SAMEORIGIN
content-security-policy: default-src 'self'; worker-src 'none'; connect-src 'self' https://product-details.mozilla.org https://treeherder.mozilla.org/api/failurecount/; font-src 'self' https://fonts.gstatic.com; img-src 'self' data: https://secure.gravatar.com https://bug1177108.bmoattachments.org/; media-src 'self' https://bug1177108.bmoattachments.org/; object-src 'none'; script-src 'self' 'nonce-U62ZIJpU6XOFChNomJa4wUygc7opa9NIV7aAV9vFxJlInNEi' 'unsafe-inline' 'unsafe-eval'; style-src 'self' 'unsafe-inline'; frame-src 'self' https://crash-stop-addon.herokuapp.com; frame-ancestors 'self'; form-action 'self' https://www.google.com/search https://github.com/login/oauth/authorize https://github.com/login https://phabricator.services.mozilla.com/ https://people.mozilla.org
date: Fri, 03 Oct 2025 06:15:39 GMT
x-served-by: cache-bom-vanm7210063-BOM, cache-bom-vanm7210068-BOM
x-cache: MISS, MISS
x-cache-hits: 0, 0
x-timer: S1759472138.606118,VS0,VE1417
vary: Accept-Encoding
1177108 - Add a keyboard shortcut to launch the eyedropper
Closed
Bug 1177108
Opened 10 years ago
Closed 2 months ago
Add a keyboard shortcut to launch the eyedropper
*
Summary:
Add a keyboard shortcut to launch the eyedropper
Categories
(DevTools :: Inspector, enhancement, P2)
Tracking
(relnote-firefox 143+, firefox143 fixed)
RESOLVED
FIXED
RESOLVED
FIXED
143 Branch
Iteration:
---
| Webcompat Score | |
| Accessibility Severity | |
| Webcompat Priority | |
| a11y-review | |
| Performance Impact |
People
(Reporter: canuckistani, Assigned: gtn)
References
(Blocks 2 open bugs)
---
---
---
Details
(Keywords: dev-doc-needed, Whiteboard: [btpp-fix-later])
---
[btpp-fix-later]
QA Whiteboard:
[qa-triage-done-c144/b143]
Has STR:
---
Change Request:
---
4
Bug Flags:
|
| ||||
Crash Data
Signature:
None
Security
(public)
This bug is publicly visible.
User Story
Attachments
(3 files, 1 obsolete file)
I would like to suggest Control E, as it's easy to reach and remember (E as in Eyedropper).
Currently this shortcut is mapped to the findbar, which has also Control K assigned.
A workaround for this missing feature is installing the keyconfig addon from: https://forums.mozillazine.org/viewtopic.php?t=72994
and applying the following lines to prefs.js in your Firefox profile folder:
//activate Developer Tools Eyedropper by pressing CTRL - E
user_pref("keyconfig.main.key_search2", "!][][");
user_pref("keyconfig.main.xxx_key2_Eyedropper", "control][E][][openEyedropper();][chrome://browser/content/browser.xul");
|
||
Comment 2•10 years ago
|
||
I'd like to work on this!
I would also recommend to add an option to the Developer Tools config screen to enable the keyboard shortcut and have it disabled by default.
Not all users understand what this EyeDropper tool is and a regular user accidentally pressing this keyboard shortcut might consider the new behavior to be a bug.
|
|
||
Comment 4•10 years ago
|
||
Please assign this to me! I'd like to solve this as my first bug!
|
||
Comment 5•10 years ago
|
||
Do we still think a keyboard shortcut for the eyedropper is going to be useful? I ask because I wonder which 'mode' would it open in.. For example:
1) if the colorpicker is opened then the eyedropper changes the selected value of the picker
2) if it isn't then it copies a color to the clipboard.
If we only supported the first use case then it may not solve the initial request (and maybe at that point we should do something like Chrome does which is have the eyedropper 'always on' when the colorpicker is opened). If we only supported the second that could be frustrating when the colorpicker *was* opened if you were wanting to actually change the selected color but it ended up just copying to the clipboard.
Flags: needinfo?(hholmes)
|
||
Comment 6•10 years ago
|
||
Being able to access the tool via the keyboard is a good idea in my opinion. While this is a separate issue, I think it might be a good idea for accessibility reasons (and a finite number of key commands available) to consider bundling some of our command button actions together into a key command. Just a tangential thought.
I'm imagining the UX of this case would go like so:
- If the colorpicker is not open, the shortcut opens the eyedropper and does not open the colorpicker.
- If the colorpicker is open, the shortcut opens the eyedropper and does not close the colorpicker.
I'm going to attach a small video of the way Sketch does this, I think they handle it well really well.
I don't think that we should do it the way Chrome does; having the eyedropper 'always on' is a little annoying in my opinion.
|
|
||
Comment 8•10 years ago
|
||
Hey Helen, thanks for the video. Your user case sounds perfect! All tools have some shortcut keys but this one. I think the shortcut key would be highly useful.
|
|
||
Comment 9•10 years ago
|
||
(In reply to knekeman from comment #1)
> I would like to suggest Control E, as it's easy to reach and remember (E as
> in Eyedropper).
Seems like this is a candidate; Brian, is there any reason why this key command wouldn't work?
(In reply to Aniket Kudale from comment #8)
> Hey Helen, thanks for the video. Your user case sounds perfect! All tools
> have some shortcut keys but this one. I think the shortcut key would be
> highly useful.
You mentioned wanting to take this bug; assuming that key command works, do you have everything you need to get started?
Flags: needinfo?(bgrinstead)
|
|
||
Comment 10•10 years ago
|
||
(In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #9)
> (In reply to knekeman from comment #1)
> > I would like to suggest Control E, as it's easy to reach and remember (E as
> > in Eyedropper).
> Seems like this is a candidate; Brian, is there any reason why this key
> command wouldn't work?
ctrl+key keyboard shortcuts are hard to get (most of them are already in use on certain platforms or by other browser features). So if nothing else ctrl+shift+e (cmd+shift+e on mac) would be safer. What key does Sketch use?
>
> (In reply to Aniket Kudale from comment #8)
> > Hey Helen, thanks for the video. Your user case sounds perfect! All tools
> > have some shortcut keys but this one. I think the shortcut key would be
> > highly useful.
> You mentioned wanting to take this bug; assuming that key command works, do
> you have everything you need to get started?
The desired behavior in Comment 6 is going to be a little complex, but certainly doable. It's worth starting - probably first by focusing on the first use case (colorpicker not open), although we should wait to land anything until both are accounted for.
Flags: needinfo?(bgrinstead)
|
|
||
Comment 11•10 years ago
|
||
(In reply to Brian Grinstead [:bgrins] from comment #10)
> (In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #9)
> > (In reply to knekeman from comment #1)
> > > I would like to suggest Control E, as it's easy to reach and remember (E as
> > > in Eyedropper).
> > Seems like this is a candidate; Brian, is there any reason why this key
> > command wouldn't work?
>
> ctrl+key keyboard shortcuts are hard to get (most of them are already in use
> on certain platforms or by other browser features). So if nothing else
> ctrl+shift+e (cmd+shift+e on mac) would be safer. What key does Sketch use?
Sketch rather inconveniently uses Ctrl+C... seems like that might be a confusing key command for devtools, if it's not already taken :( Let's go with your ctrl+shift+e (cmd+shift+e on mac) suggestion.
> The desired behavior in Comment 6 is going to be a little complex, but
> certainly doable. It's worth starting - probably first by focusing on the
> first use case (colorpicker not open), although we should wait to land
> anything until both are accounted for.
Oh no! Sorry for always complicating things...
|
|
||
Comment 12•10 years ago
|
||
(In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #9)
> (In reply to knekeman from comment #1)
> > I would like to suggest Control E, as it's easy to reach and remember (E as
> > in Eyedropper).
> Seems like this is a candidate; Brian, is there any reason why this key
> command wouldn't work?
>
> (In reply to Aniket Kudale from comment #8)
> > Hey Helen, thanks for the video. Your user case sounds perfect! All tools
> > have some shortcut keys but this one. I think the shortcut key would be
> > highly useful.
> You mentioned wanting to take this bug; assuming that key command works, do
> you have everything you need to get started?
Hi, Yes I have dev environment ready! waiting for the assignment of this bug to me. Thanks
|
|
||
Updated•10 years ago
|
Assignee: nobody → aniket.kudale
|
|
||
Comment 13•10 years ago
|
||
(In reply to Aniket Kudale [:aniketk] from comment #12)
> Hi, Yes I have dev environment ready! waiting for the assignment of this bug
> to me. Thanks
All right, you're good to go!
|
|
||
Comment 14•10 years ago
|
||
(In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #11)
> (In reply to Brian Grinstead [:bgrins] from comment #10)
> > (In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #9)
> > > (In reply to knekeman from comment #1)
> > > > I would like to suggest Control E, as it's easy to reach and remember (E as
> > > > in Eyedropper).
> > > Seems like this is a candidate; Brian, is there any reason why this key
> > > command wouldn't work?
> >
> > ctrl+key keyboard shortcuts are hard to get (most of them are already in use
> > on certain platforms or by other browser features). So if nothing else
> > ctrl+shift+e (cmd+shift+e on mac) would be safer. What key does Sketch use?
> Sketch rather inconveniently uses Ctrl+C... seems like that might be a
> confusing key command for devtools, if it's not already taken :( Let's go
> with your ctrl+shift+e (cmd+shift+e on mac) suggestion.
We cannot go for 'ctrl+shift+e', it is already been used for grouping tabs!
I think 'ctrl+shift+z' is best solution for this, since it isn't been used for any other purpose!
I'm attaching the patch file with changes made.
- Cheers!!
|
|
||
Comment 16•10 years ago
|
||
Comment on attachment 8715250 [details] [diff] [review]
Added shortcut keys to open Eyedropper.
Switching the review request to Brian—I think he's more familiar with the code.
Thanks for the patch!
Attachment #8715250 -
Flags: review?(dcamp) → review?(bgrinstead)
|
|
||
Updated•10 years ago
|
Component: Developer Tools → Developer Tools: Inspector
|
|
||
Comment 17•10 years ago
|
||
Comment on attachment 8715250 [details] [diff] [review]
Added shortcut keys to open Eyedropper.
Review of attachment 8715250 [details] [diff] [review]:
-----------------------------------------------------------------
This is a good start. We need a couple of other pieces here:
When the colorpicker is opened we want to trigger the eyedropper within the colorpicker as in Comment 6 (i.e. open a new tab data:text/html,<body style='color:orange'>hi</body>, inspect the body, then open up the colorpicker on the orange swatch). There's an object here with an _openEyeDropper function that can be called: https://dxr.mozilla.org/mozilla-central/source/devtools/client/shared/widgets/Tooltip.js#1127.
As for how to pipe these two things together, we could either go from browser.js/openEyedropper level and find an active colorpicker or we could add the a key shortcut in the tooltip and prevent default behavior. I'm inclined to go with the latter - so we'd need to then bind to the keydown handler in the Tooltip the colorpicker popup and preventDefault on that key so we don't also trigger the browser's command. If that works, then we can expose a duplicate key through a properties file like https://dxr.mozilla.org/mozilla-central/source/devtools/client/locales/en-US/eyedropper.properties.
Attachment #8715250 -
Flags: review?(bgrinstead) → feedback+
|
|
||
Updated•10 years ago
|
Status: NEW → ASSIGNED
|
|
||
Comment 18•10 years ago
|
||
I don't understand this case - triggering the eyedropper within the colorpicker. Do you mean when I open the colorpicker on color swatch and press the shortcut keys, it should trigger eyedropper? I am getting a little confused here!
|
|
||
Comment 19•10 years ago
|
||
(In reply to Aniket Kudale [:aniketk] from comment #18)
> I don't understand this case - triggering the eyedropper within the
> colorpicker. Do you mean when I open the colorpicker on color swatch and
> press the shortcut keys, it should trigger eyedropper? I am getting a little
> confused here!
Right now in the colorpicker if you press the 'eyedropper' button then the colorpicker closes and the eyedropper opens. Then when you select a color from the page instead of it copying to the clipboard, it sets the current color value on the CSS property. So if the eyedropper is triggered from a keyboard shortcut while the colorpicker is opened, then it should have the same behavior (as if the button was pressed)
|
||
Comment 20•9 years ago
|
||
Inspector bug triage. Filter on CLIMBING SHOES.
Not sure about the CTRL+SHIFT+Z shortcut: it is the alternate shortcut for "Redo" (even if there is also CTRL+Y). Just tested on FF Windows and OSX and indeed CTRL+SHIFT+Z and CTRL+Y are both mapped to "Redo".
FWIW, CTRL+SHIFT+I seems available.
Aniket: Are you still working on this bug?
Severity: normal → enhancement
Flags: needinfo?(aniket.kudale)
Priority: -- → P3
Whiteboard: [btpp-fix-later]
|
|
||
Comment 21•9 years ago
|
||
CTRL+SHIFT+I is for Toggle Tools on Windows.
I think we need to finalize the shortcut keys combination suitable for all platform.
Once finalized, I'll update and upload the patch!
|
|
||
Updated•9 years ago
|
Flags: needinfo?(aniket.kudale)
|
||
Comment 23•9 years ago
|
||
Aniket, what's the status of this bug? Are you still working on it? If not, you should remove yourself as assignee, so somebody else can jump in.
Sebastian
Flags: needinfo?(aniket.kudale)
|
|
||
Comment 24•9 years ago
|
||
Hi,
We still need to decide what keys combination we need for eyedropper.
Once finalized, I'll update the patch.
Cheers,
Aniket Kudale
aniket.co
|
|
||
Comment 25•9 years ago
|
||
(In reply to Aniket Kudale [:aniketk] from comment #14)
> (In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #11)
> > (In reply to Brian Grinstead [:bgrins] from comment #10)
> > > (In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #9)
> > > > (In reply to knekeman from comment #1)
> > > > > I would like to suggest Control E, as it's easy to reach and remember (E as
> > > > > in Eyedropper).
> > > > Seems like this is a candidate; Brian, is there any reason why this key
> > > > command wouldn't work?
> > >
> > > ctrl+key keyboard shortcuts are hard to get (most of them are already in use
> > > on certain platforms or by other browser features). So if nothing else
> > > ctrl+shift+e (cmd+shift+e on mac) would be safer. What key does Sketch use?
> > Sketch rather inconveniently uses Ctrl+C... seems like that might be a
> > confusing key command for devtools, if it's not already taken :( Let's go
> > with your ctrl+shift+e (cmd+shift+e on mac) suggestion.
>
> We cannot go for 'ctrl+shift+e', it is already been used for grouping tabs!
Is ctrl+shift+e still reserved? The tabs group feature has been removed, so maybe we can claim ctrl+shift+e now: https://support.mozilla.org/en-US/kb/tab-groups-removal
|
|
||
Comment 26•9 years ago
|
||
(In reply to Brian Grinstead [:bgrins] from comment #25)
> (In reply to Aniket Kudale [:aniketk] from comment #14)
> > (In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #11)
> > > (In reply to Brian Grinstead [:bgrins] from comment #10)
> > > > (In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #9)
> > > > > (In reply to knekeman from comment #1)
> > > > > > I would like to suggest Control E, as it's easy to reach and remember (E as
> > > > > > in Eyedropper).
> > > > > Seems like this is a candidate; Brian, is there any reason why this key
> > > > > command wouldn't work?
> > > >
> > > > ctrl+key keyboard shortcuts are hard to get (most of them are already in use
> > > > on certain platforms or by other browser features). So if nothing else
> > > > ctrl+shift+e (cmd+shift+e on mac) would be safer. What key does Sketch use?
> > > Sketch rather inconveniently uses Ctrl+C... seems like that might be a
> > > confusing key command for devtools, if it's not already taken :( Let's go
> > > with your ctrl+shift+e (cmd+shift+e on mac) suggestion.
> >
> > We cannot go for 'ctrl+shift+e', it is already been used for grouping tabs!
>
> Is ctrl+shift+e still reserved? The tabs group feature has been removed, so
> maybe we can claim ctrl+shift+e now:
> https://support.mozilla.org/en-US/kb/tab-groups-removal
It still exists as extension, though. So, there would still be a conflict when it is installed. How about Ctrl+Alt+E?
Sebastian
|
||
Comment 27•9 years ago
|
||
Wow! I'm very happy that someone now look after that! I voted for that month ago @ the uservoice.com site but sadly until now nothing was done. :(
CTRL + Shift + Z would be my suggestion! In case it`s used by another feature I think there are certainly enough other free keys to manage this. ;)
Another suggestion is to implement a icon for the eyedropper into the "Additional Tools and Features" window where users can customize their toolbar. In this case everyone who really needs it could then customize his toolbar with it to get a quicker access!
You must know that a simple shortcut reduces the clicks to get the eyedropper significantly! A eyedropper is really useful thing for e.g. a webdesigners who is working on his design and searching for some inspiration on the web. I use the colorpicker very, very often e.g while developing a web template... and the fact that I always have to press ALT to open the FF menu, than move down to dev tools to get the eydropper is really anoying! In the Developer Edition of FF there is at least the option to activate the tiny little eydropper icon in the panel which is much much better than crawl through the Extras menu! Until now I don't know why this feature is only implemented in the Developer Edition of FF?
So I hope that this feature incl. a simple shortcut will soon be implemented in the new version...and thx in advance to all devs who are working on a solution... :)
|
||
Updated•9 years ago
|
Blocks: top-inspector-bugs
|
|
||
Updated•9 years ago
|
Priority: P3 → P2
|
|
||
Comment 28•9 years ago
•
|
||
Aniket, there were a few suggestions for the shortcut in the meantime:
Ctrl+Shift+E (conflicts with the Tab Groups extension)
Ctrl+Alt+E
Ctrl+Shift+Z (conflicts with redoing a deleted node in the Inspector)
If shortcuts with E are a problem, maybe one with C (for *C*olor picker) like Ctrl+Alt+C may work or Ctrl+Shift+Y (for e*Y*edropper).
So, I guess it's up to you to pick one.
Sebastian
|
|
||
Comment 29•8 years ago
|
||
As Aniket wasn't active on Bugzilla for more than half a year now, I think it's safe to unassign this bug again to give somebody else the chance to add this.
Regarding my previous comment, I'd vote for Ctrl+Shift+Y, or, as Tab Groups won't work anymore with Firefox 57, Ctrl+Shift+E. Ctrl+Alt+E actually conflicts with the Euro sign.
Sebastian
Assignee: aniket.kudale → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(aniket.kudale)
|
|
||
Comment 30•8 years ago
|
||
Hey guys, what's up? Two years have gone now since this discussion was sartet... and you still talk about which shortcut you might gonna use! This is a joke, isn't it!? On the very top of this thread the best solution was suggested by Acheron:
(In reply to Acheron from comment #3)
> I would also recommend to add an option to the Developer Tools config screen
> to enable the keyboard shortcut and have it disabled by default.
>
> Not all users understand what this EyeDropper tool is and a regular user
> accidentally pressing this keyboard shortcut might consider the new behavior
> to be a bug.
To not assign another shortcut... why you don't give the user the option to choose their own shortcut!?!?!?!?!??!?!?!
|
|
||
Comment 31•8 years ago
|
||
(In reply to mitzi.akhavan from comment #30)
> Hey guys, what's up? Two years have gone now since this discussion was
> sartet... and you still talk about which shortcut you might gonna use! This
> is a joke, isn't it!? On the very top of this thread the best solution was
> suggested by Acheron:
>
> (In reply to Acheron from comment #3)
> > I would also recommend to add an option to the Developer Tools config screen
> > to enable the keyboard shortcut and have it disabled by default.
> >
> > Not all users understand what this EyeDropper tool is and a regular user
> > accidentally pressing this keyboard shortcut might consider the new behavior
> > to be a bug.
>
> To not assign another shortcut... why you don't give the user the option to
> choose their own shortcut!?!?!?!?!??!?!?!
Comments such as "this is a joke" or "why don't you just ..." won't help get this bug resolved, and will certainly not attract people to fix this bug.
Please be constructive and, if you have JS coding skills, please consider contributing yourself to this bug: https://devtools-html.github.io/
I, too, would love it if things were as simple as "just fixing" everything. Unfortunately that's not how things work.
|
|
||
Comment 32•8 years ago
|
||
Hello Patrik,
sorry for my rude statement. I dont want to put sombodys's nose out of joint. I only wanted to say that now 2 years have gone and for me it's not really understandable that people still discuss about the right shortcut. But thats only my point of view. Hope you still can understand it? Believe me... I really appreciate all the developer work and if I could easily write JS code like you all I surely would help you out or at least I would build my own FF addon to get my own shortcut. The best thing now I can do is to bring something into action again..., sorry.
Kind regards,
Mitzi
|
||
Updated•7 years ago
|
Product: Firefox → DevTools
|
||
Updated•7 years ago
|
status-firefox41:
affected → ---
Whiteboard: [btpp-fix-later] → [btpp-fix-later][dt-q]
|
|
||
Updated•7 years ago
|
Whiteboard: [btpp-fix-later][dt-q] → [btpp-fix-later]
|
|
||
Updated•6 years ago
|
No longer blocks: top-inspector-bugs
|
||
Comment 33•6 years ago
|
||
Any update on this?
|
||
Comment 34•4 years ago
|
||
Hello from the future :)
Maybe we can mark this as good first bug to attract someone?
Until then one can use "Alt + T + B + Y" :D
|
||
Comment 35•4 years ago
|
||
| metoo | ||
| Comment hidden (metoo) |
Will this ever be brought back?
|
|
||
Updated•3 years ago
|
Severity: normal → S3
|
||
Comment 36•2 years ago
|
||
| advocacy | ||
| Comment hidden (advocacy) |
It would be nice to have this feature for those who do webdev work on Firefox
|
||
Updated•1 years ago
|
Blocks: devtools-shortcuts
|
Assignee | |
Comment 37•3 months ago
|
||
Hi, is this still an issue ? I've got some spare time and I'm looking for some easy issues to start with :)
|
|
||
Comment 38•3 months ago
|
||
Yep, there's no shortcut for eyedropper at the moment.
|
|
||
Comment 39•3 months ago
|
||
Gaétan, I've assigned the bug to you now! Regarding my previous comment about which shortcut to choose, I'd still say Ctrl+Shift+Y. Ctrl+Shift+E is now taken by the Network panel and Ctrl+Alt+E is for the Euro sign.
Sebastian
Assignee: nobody → gtn
|
|
Assignee | |
Comment 40•3 months ago
|
||
I have a first version working (still need tests and a bit of cleanup), but it only works when the inspector is opened, is that the desired behavior ? Or should I make it work even when it's not opened ? Because then Ctrl+Shift+Y is already the shortcut for the library.
|
|
||
Comment 41•3 months ago
|
||
it only works when the inspector is opened, is that the desired behavior ?
DevTools already registers too many shortcuts, so it should only work when the inspector is opened and focused.
|
|
||
Comment 42•3 months ago
|
||
The desired behavior is so it would not depend on the inspector. The idea is akin to helping artists pull out a desired color from an image they see in the browser.
|
|
||
Comment 43•3 months ago
|
||
(In reply to tsukiyomaru0 from comment #42)
The desired behavior is so it would not depend on the inspector. The idea is akin to helping artists pull out a desired color from an image they see in the browser.
Maybe an addon would be a better fit for this specific request? https://addons.mozilla.org/en-US/firefox/addon/colorzilla/ seems to do what you expect.
As I said, DevTools already registers too many global shortcuts. A patch adding a new shortcut which doesn't depend on DevTools being open will not be accepted.
|
|
Assignee | |
Comment 44•3 months ago
|
||
So, I write some tests and submit a patch, making it available only when the inspector is opened. Does that seem good to you ?
|
|
||
Comment 45•3 months ago
|
||
(In reply to Gaétan Fine from comment #44)
So, I write some tests and submit a patch, making it available only when the inspector is opened. Does that seem good to you ?
Sounds good, thanks!
|
|
Assignee | |
Comment 46•2 months ago
|
||
|
|
Assignee | |
Comment 47•2 months ago
|
||
Attached file
Bug 1177108 - [devtools] Merge the eyedropper+searchbox shortcut. r=jdescottes! (obsolete)
— Details
|
||
Updated•2 months ago
|
Attachment #9503248 -
Attachment is obsolete: true
|
||
Comment 48•2 months ago
|
||
Pushed by jdescottes@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/574a9c54e77c
https://hg.mozilla.org/integration/autoland/rev/2ee633308b91
[devtools] Add shortcut to eyedropper. r=jdescottes,devtools-reviewers
|
|
||
Comment 49•2 months ago
|
||
Pushed by amarc@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/20dc0130e0bf
https://hg.mozilla.org/integration/autoland/rev/427bc623f84e
Revert "Bug 1177108 - [devtools] Add shortcut to eyedropper. r=jdescottes,devtools-reviewers" for causing dt failures @ browser_inspector_highlighter-eyedropper-keyboard-shortcut.js
|
|
||
Comment 51•2 months ago
|
||
Ah slightly bad timing, this landed right after Bug 1980576 landed, which introduced breaking changes for the test.
Flags: needinfo?(gtn)
|
|
||
Comment 52•2 months ago
|
||
Pushed by jdescottes@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/fa7bb621eb26
https://hg.mozilla.org/integration/autoland/rev/1e8cb4dd91bf
[devtools] Add shortcut to eyedropper. r=jdescottes,devtools-reviewers
|
||
Comment 53•2 months ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 2 months ago
status-firefox143:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 143 Branch
|
||
Updated•1 month ago
|
QA Whiteboard: [qa-triage-done-c144/b143]
|
|
||
Comment 54•16 days ago
|
||
A bit late, though maybe it can be added to the release notes at https://www.firefox.com/en-US/firefox/143.0/releasenotes/ and maybe https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/143.
[Why is this notable]: Long requested webdev feature
[Affects Firefox for Android]: no
[Suggested wording]: When the Inspector is focused, you can now launch the eyedropper to pick a color via the keyboard shortcut Ctrl+Shift+Y.
[Links (documentation, blog post, etc)]: none I know of, should probably be added to https://firefox-source-docs.mozilla.org/devtools-user/page_inspector/how_to/inspect_and_select_colors/index.html and https://firefox-source-docs.mozilla.org/devtools-user/keyboard_shortcuts/index.html.
relnote-firefox:
--- → ?
Keywords: dev-doc-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•