Closed
Bug 1132950
Opened 9 years ago
Closed 9 years ago
Sorting direction markers are staying when another column is sorted
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
FIXED
Firefox 38
Tracking | Status | |
---|---|---|
firefox37 | --- | unaffected |
firefox38 | --- | fixed |
People
(Reporter: alice0775, Assigned: ntim)
References
Details
(Keywords: ux-mode-error)
Attachments
(2 files, 1 obsolete file)
87.55 KB,
image/png
|
Details | |
1.06 KB,
patch
|
ntim
:
review+
|
Details | Diff | Splinter Review |
Steps To Reproduce: 1. Open about:config 2. Click "Preference name" tree header to change sort direction --- Sorting direction marker appears on "Preference name" column as expected 3. Click another tree header(e.g. "Status") to change sort direction --- Sorting direction marker appears on "Status" column as expected However, Sorting direction marker is still shown on "Preference name" column Actual Results: Sorting direction markers are staying when another column is sorted Expected Results: Sorting direction markers should only be shown on column header that sorted
Reporter | ||
Comment 1•9 years ago
|
||
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(richard.marti)
Comment 2•9 years ago
|
||
It's not a bug directly introduced by bug 1125636. The bug comes from https://dxr.mozilla.org/mozilla-central/source/toolkit/themes/shared/in-content/common.inc.css#686 There are two solutions: 1. Change the CSS code to not show the arrow when the empty attribute "sortDirection" is set. 2. Check where the "sortDirection" is set in JS and not removed when a other column is the active sort column. For 1. I could fix this, but I think 2. would be cleaner but I'm not a JS specialist.
Flags: needinfo?(richard.marti)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → ntim007
Status: NEW → ASSIGNED
OS: Windows 7 → All
Hardware: x86_64 → All
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8564319 -
Flags: review?(jaws)
Comment 4•9 years ago
|
||
Comment on attachment 8564319 [details] [diff] [review] Patch Review of attachment 8564319 [details] [diff] [review]: ----------------------------------------------------------------- Looks like we could do with just the config.js changes. Any reason that we need to make the changes for common.inc.css? Presence of an attribute in XUL-land is pretty common to mean that it should be enabled.
Assignee | ||
Comment 5•9 years ago
|
||
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #4) > Comment on attachment 8564319 [details] [diff] [review] > Patch > > Review of attachment 8564319 [details] [diff] [review]: > ----------------------------------------------------------------- > > Looks like we could do with just the config.js changes. Any reason that we > need to make the changes for common.inc.css? Presence of an attribute in > XUL-land is pretty common to mean that it should be enabled. I guess so, I just want to prevent this in the future.
Comment 6•9 years ago
|
||
Comment on attachment 8564319 [details] [diff] [review] Patch Review of attachment 8564319 [details] [diff] [review]: ----------------------------------------------------------------- Yeah, not necessary though. Let's keep this change small and only do the config.js change. Also, we shouldn't use :-moz-any() as the right-most selector: https://developer.mozilla.org/en-US/docs/Web/CSS/:any#Issues_with_performance_and_specificity r=me for only the config.js changes.
Attachment #8564319 -
Flags: review?(jaws) → review+
Assignee | ||
Comment 7•9 years ago
|
||
Removed the CSS changes.
Attachment #8564319 -
Attachment is obsolete: true
Attachment #8564582 -
Flags: review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 8•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/2c5db2a83e34
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Comment 9•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2c5db2a83e34
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 38
Comment 10•9 years ago
|
||
Mistakenly filed against Firefox 38 and should be instead 38 Branch. Sorry for the spam. dkl
Version: Firefox 38 → 38 Branch
Updated•9 years ago
|
QA Whiteboard: [good first verify]
You need to log in
before you can comment on or make changes to this bug.
Description
•