The timeline has been heard described as "info-porn". IOW, while fun and pleasing and sometimes useful, it's maybe not always that essential, and some people may not value it enough to give it prime above-the-fold space all the time. One idea is to have a toggle feature (glyph, words, whatever) that allows one to collapse the toolbar to use a very small amount of space (maybe 0 vertical space), and persist that choice across queries.
We should probably also add the "zoom out" affordance at the same time... right now, once you select a time range, there's no/limited going back. (You can get back out to the year because it is displayed, but you can't zoom out further than that.)
Created attachment 399877 [details] timeline hide mockup I think if we offer the timeline as a button off to the right where we grow the space for the widget people can show / hide it as needed.
Created attachment 400241 [details] [diff] [review] patch v1 This works for me. I made up the icon from your mockup, and I didn't make the borders, for a variety of reasons (hard to figure out how to deal with wide windows elegantly, not sure we want that bold a border, didn't know if you just wanted the border on hover, etc.). I think the code review and ui review can be quite parallel here.
Comment on attachment 400241 [details] [diff] [review] patch v1 = UI input, yielding to clarkbw where there is debate: The hovered timeline has no whitespace between it and the AND/OR toggle button. I think there should be. I find the animation effect's involvement of the x-axis distracting. I think a vertical rollup would be more straightforward. = Logic The timeline widget's state is inconsistent when initialized collapsed. It is hidden, but _timelineShown is never set to false, so the next time you try and toggle it, you're just hiding it again. I would suggest losing the ugly preference branch boilerplate and just using Application.prefs intead: https://developer.mozilla.org/en/Toolkit_API/extIPreferenceBranch r=asuth with the logic changes, UI changes at your/clarkbw's discretion.
Created attachment 400336 [details] [diff] [review] revised patch fixed the logic & code bugs. dealt with the overlap w/ the and/or box. I can't repro any horizontal animation, so I'm a bit stumped w/ that one.
Created attachment 400538 [details] [diff] [review] fixed patch fixed the horizontal glitch (having to do w/ style tweaks happening on slideUp, and margins), centered the timeline within the 'results' column, and changed the sizing process for the timeline to work even when 'hidden' therefore avoiding rebuild on toggling.
Comment on attachment 400538 [details] [diff] [review] fixed patch this patch failed to apply for me
Created attachment 400560 [details] [diff] [review] De-bitrotted patch debitrotted
Comment on attachment 400560 [details] [diff] [review] De-bitrotted patch just change the pref to default to the timeline being collapsed. ui-r+ with that
Created attachment 400562 [details] [diff] [review] patch w/ other pref value your wish is my command.
It not really nice for .css living in /content/ to refer to images living in /skin/. One should really define the style rules that refer to images in /skin/ in .css living in /skin/.