Closed
Bug 656928
Opened 13 years ago
Closed 13 years ago
Make exhibit slider facet work with standards mode
Categories
(Webtools Graveyard :: Elmo, defect)
Webtools Graveyard
Elmo
Tracking
(Not tracked)
VERIFIED
FIXED
1.2
People
(Reporter: Pike, Assigned: Pike)
References
Details
Attachments
(1 file)
2.71 KB,
patch
|
Pike
:
review+
|
Details | Diff | Splinter Review |
There's a fix in upstream exhibit to make the slider facet work in standards mode. We should downstream that, just a cherry pick. Needs to land in slider.js and all bundles.
Assignee | ||
Comment 1•13 years ago
|
||
Upstream ticket is http://code.google.com/p/simile-widgets/issues/detail?id=283 (which I filed, go figure), fix is http://code.google.com/p/simile-widgets/source/diff?spec=svn2088&r=2088&format=side&path=/exhibit/trunk/src/webapp/api/scripts/ui/facets/slider.js
Assignee | ||
Comment 2•13 years ago
|
||
updating the slider and bundles. The differences in quoting are intentional, as that's what the code does in exhibit, apparenlty. Got r=stas on irc.
Attachment #532248 -
Flags: review+
Assignee | ||
Comment 3•13 years ago
|
||
https://github.com/mozilla/elmo/commit/3c1a55f872aaeb89a358e259d31bf923c76b4f0d, FIXED.
Assignee: nobody → l10n
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.2
Comment 4•13 years ago
|
||
How do I test this in "standards mode?" -- thanks!
Assignee | ||
Comment 5•13 years ago
|
||
If it works, it's VERIFIED. The culprit was that the page went from quirks mode to strict mode with a template change, and regressed the existing behaviour. Works on https://l10n-stage-sj.mozilla.org/shipping/dashboard?locale=pl (it's the slider for missing strings which you can shuffle around), which works and shows the histogram, so marking VERIFIED.
Status: RESOLVED → VERIFIED
Updated•4 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•