- 04 Aug, 2014 1 commit
-
-
Randall Leeds authored
I missed this in b67b2964b7a1fd1271be650ea02cf967fd0e17a3
-
- 02 Aug, 2014 3 commits
-
-
Randall Leeds authored
Clean up scrolling and infinite stream
-
gergely-ujvari authored
-
Randall Leeds authored
Simplify our efforts around scrolling and fix the scrollbars on the stream. - Scrollbar had crept inside the content on a previous change such that scroll bars for the stream were not all the way to the edge of the screen. Refactoring the duties of #wrapper and .content. - Drop the mousewheel trapping. I tried and looked and researched and we can't do this reliably on touch or anything else. It didn't seem necessary or worth the dependency on jquery.mousewheel or even obviously a good experience. Not worth the code. - Fix infinite scroll to use its directive element rather than the $window, make its code less jquery, and give it a buffer to smooth the experience a bit more. Fix #1360
-
- 01 Aug, 2014 19 commits
-
-
Randall Leeds authored
Fix #1370
-
Randall Leeds authored
Add directive for providing saving updates to buttons
-
Randall Leeds authored
Using transclusion allows for the directive to be set on an element with non-trivial content and have that content nested inside the status-button wrapper spans, but pre-bound to its lexical markup scope. Not sure we'll need it here, but I also wanted to see how one might approximate 'replace' now that it's deprecated. The answer is by not replacing but just adding the template you want after the passed placeholder comment node.
-
Aron Carroll authored
-
Aron Carroll authored
-
Aron Carroll authored
-
Aron Carroll authored
-
Aron Carroll authored
This adds loading/success messages to a button for use with async actions such as changing account information.
-
gergely-ujvari authored
Search callback routing cleanup
-
gergely-ujvari authored
The filtered copy list was not assigned back to the variable and accidently we were comparing the value.length instead of the filter.terms.length to check whether all tags have matched or not.
-
Randall Leeds authored
Improve usability of sheet tabs
-
gergely-ujvari authored
-
gergely-ujvari authored
-
Randall Leeds authored
This unifies the stream and page_search query parameter usage. Both now use the 'q' parameter. Additionally, the search string is passed to `simpleSearch` directive and the `QueryParser` instead of an object with a "query" or "q" key. The clump of conditionals in the route update event handler is removed. Instead, the viewer and page search redirect to one another and the stream can be totally ignorant of the involvement of the location. The views are updated to use ?q= for /u and /t shortcuts.
-
Randall Leeds authored
-
Randall Leeds authored
Remove the drop shadow from the highlight states
-
Aron Carroll authored
-
Aron Carroll authored
Previously the active tab would be red, which could be confused with our red anchor text and make the other tab look disabled. The new design makes all text gray but adds an "active" border below the button.
-
Aron Carroll authored
-
- 31 Jul, 2014 10 commits
-
-
Aron Carroll authored
-
Aron Carroll authored
-
Randall Leeds authored
Simplified search
-
Randall Leeds authored
Update from error styles based on feedback
-
Aron Carroll authored
Based on feedback from Dan that the plain red text is hard to distinguish from hyper-links.
-
Aron Carroll authored
Tweak the HTML and CSS of the new search input.
-
Randall Leeds authored
-
Randall Leeds authored
Besides the line height hack for FF kill the rest of the px related calculation and do everything in nice, pretty ems.
-
Randall Leeds authored
The trick used here was designed to honor the font setting in a user's browser by assuming a typical default of 16px and computing the base font size in ems relative to that. The problem is that even if everything is sized in ems there are sometimes aggressive user agent styles that complicate things by restoring the system font. It turns out it fixes the various things I've noticed in my FF this way.
-
Randall Leeds authored
-
- 30 Jul, 2014 1 commit
-
-
Aron Carroll authored
This may fix the text issue displayed in FF with custom display settings. https://github.com/hypothesis/h/pull/1326#issuecomment-50655944
-
- 29 Jul, 2014 6 commits
-
-
Aron Carroll authored
* Remove the background color. * Borders and icon darken when the input is in focus. * Adds the placeholder text back.
-
gergely-ujvari authored
-
gergely-ujvari authored
-
Aron Carroll authored
-
Aron Carroll authored
-
gergely-ujvari authored
-