Closed
Bug 1462237
Opened 6 years ago
Closed 6 years ago
Prevent text selection on the application panel
Categories
(DevTools :: Application Panel, enhancement, P3)
Tracking
(firefox62 fixed)
RESOLVED
FIXED
Firefox 62
Tracking | Status | |
---|---|---|
firefox62 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
Details
Attachments
(1 file)
In most screens of the application panel, we shouldn't allow text selection.
The spots where text selection might still be useful are:
- scope
- source url
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Priority: -- → P3
Comment hidden (mozreview-request) |
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8980584 [details]
Bug 1462237 - Prevent text selection in application panel;
https://reviewboard.mozilla.org/r/246734/#review253296
A question about the color choice. Everything else looks good to me! Thanks :)
::: devtools/client/application/src/components/App.css:33
(Diff revision 1)
> }
>
> a.disabled,
> a.disabled:hover,
> a.disabled:visited {
> - opacity: 0.5 !important;
> + color: var(--theme-body-color-inactive) !important;
Do we have UX input about this? With `--theme-body-color-inactive` disabled links look exactly the same as regular text, and I'm not sure if this is what we'd want
Assignee | ||
Comment 3•6 years ago
|
||
mozreview-review-reply |
Comment on attachment 8980584 [details]
Bug 1462237 - Prevent text selection in application panel;
https://reviewboard.mozilla.org/r/246734/#review253296
Thanks for the review!
> Do we have UX input about this? With `--theme-body-color-inactive` disabled links look exactly the same as regular text, and I'm not sure if this is what we'd want
Oops sorry about this, this was not supposed to be part of this diff.
I have a pending question to victoria about this in https://bugzilla.mozilla.org/show_bug.cgi?id=1460209
I'll drop it from the diff.
Comment hidden (mozreview-request) |
Comment 5•6 years ago
|
||
mozreview-review |
Comment on attachment 8980584 [details]
Bug 1462237 - Prevent text selection in application panel;
https://reviewboard.mozilla.org/r/246734/#review253464
Attachment #8980584 -
Flags: review?(balbeza) → review+
Comment hidden (mozreview-request) |
Comment 7•6 years ago
|
||
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.
hg error in cmd: hg rebase -s f54639855b83a32fe43add241a6b7ab6688b6eb3 -d a31957c18991: rebasing 465534:f54639855b83 "Bug 1462237 - Prevent text selection in application panel;r=ladybenko" (tip)
merging devtools/client/application/src/components/Worker.css
merging devtools/client/application/src/components/Worker.js
warning: conflicts while merging devtools/client/application/src/components/Worker.css! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment 8•6 years ago
|
||
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.
hg error in cmd: hg rebase -s f54639855b83a32fe43add241a6b7ab6688b6eb3 -d a31957c18991: rebasing 465534:f54639855b83 "Bug 1462237 - Prevent text selection in application panel;r=ladybenko" (tip)
merging devtools/client/application/src/components/Worker.css
merging devtools/client/application/src/components/Worker.js
warning: conflicts while merging devtools/client/application/src/components/Worker.css! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment hidden (mozreview-request) |
Comment 10•6 years ago
|
||
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/879d8652dde3
Prevent text selection in application panel;r=ladybenko
Comment 11•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 62
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•