- 25 Aug, 2015 2 commits
-
-
Randall Leeds authored
This lets us require Promise without polluting the global namespace. Additionally, the core-js implementation has immediacy hacks that will make it more performant for IE. This also caught some issues with our async tests. And hey, it's about 700 bytes smaller, too!
-
Randall Leeds authored
-
- 24 Aug, 2015 1 commit
-
-
Randall Leeds authored
This fixes the dialogs to have the same space around them as the main content. It's a bit like re-introducing the #wrapper, but with a class this time that's re-used by the topbar for sizing its content.
-
- 20 Aug, 2015 2 commits
-
-
Randall Leeds authored
This is a little bit confusing, but it is better this way. It's confusing because webassets doesn't set the compass image directory, so the paths still have an "images/" prefix. However, the cssrewrite filter uses the output URL from webassets and that will be an absolute URL at request time. The compass filter uses only the base URL from the environment configuration, which is relative. Letting compass do the rewriting therefore results in merged compass files having correct, but relative, paths regardless of whether the assets are generated at the command line or at request time. This should make it possible to pre- compile the assets before the first request without generating bad URLs, and to virtual host the application under multiple URLs.
-
Randall Leeds authored
In order to upgrade clean-css it is necessary to import the filter definition from the future version of webassets. This version takes extra arguments in the filter configuration, allowing us to disable the advanced optimizations that became the default in 3.0+ but that breaks some of our styles.
-
- 19 Aug, 2015 12 commits
-
-
Randall Leeds authored
-
conordelahunty authored
Fix #2452
-
Randall Leeds authored
-
Randall Leeds authored
-
Randall Leeds authored
-
Randall Leeds authored
-
Randall Leeds authored
-
Randall Leeds authored
-
Randall Leeds authored
-
Randall Leeds authored
This lets the top bar set a font size for its inner content while still matching the padding to the rest of the page.
-
Randall Leeds authored
-
Randall Leeds authored
This is much neater. Since the scale origin is the center of the icon this will scale around the center of the line, avoiding any messy vertical alignment nonsense. This works much better than vertical-align middle and font-size because vertical-align middle puts the center of the element at the x-height, rather than the middle of the line. This is much cleaner.
-
- 18 Aug, 2015 6 commits
-
-
Randall Leeds authored
Lay out the search bar and the top bar using flexbox because the vertical centering is much better and easier using flexbox instead of floats and line heights. As long as we're using only features easily supported in the 2012 and final spec the -ms prefix gets us IE10. All other browsers we care about should be supported already by the compass mixins.
-
Randall Leeds authored
- Use .btn wherever we use .btn-clean - No border on .btn-clean
-
Randall Leeds authored
This is necessary for true vertical alignment. See http://christopheraue.net/2014/03/05/vertical-align/
-
Randall Leeds authored
-
Randall Leeds authored
- Convert some units to ems - Remove extraneous rules - Remove extraneous integer parts of floats
-
Jake Hartnell authored
-
- 17 Aug, 2015 2 commits
-
-
Jake Hartnell authored
-
Jake Hartnell authored
Fix https://github.com/hypothesis/h/issues/2429
-
- 14 Aug, 2015 6 commits
-
-
Randall Leeds authored
-
Randall Leeds authored
-
Randall Leeds authored
-
Randall Leeds authored
Oops
-
Randall Leeds authored
Replace the use of the websocket for scrollback with a simple pagination scheme.
-
Randall Leeds authored
Rather than having a fixed body that scrolls, let the body take a natural height and rewrite the infinite scroll handler to watch the global scroll. The infinite scroll directive is now smarter and has tests.
-
- 13 Aug, 2015 4 commits
-
-
Randall Leeds authored
New version released has a my fix for height 100% html and body tags. Restores scroll-to-annotation in PDFs.
-
Randall Leeds authored
This makes the code easier to understand because it makes explicit the relationship to the line height of the label.
-
Randall Leeds authored
Close #2421
-
Randall Leeds authored
This change simplifies the AnnotationSync events and improves the frame symmetry by giving the guest its own AnnotationSync setup function, alongside its AnnotationUISync setup, that mirrors the way its handled in the widget. Previously, the `loadAnnotations` and `deleteAnnotations` events were treated specially by AnnotationSync so that it didn't cause endless loops. Instead, fully embrace the fact that the guest is not responsible for loading or deleting annotations. - Remove the setupAnnotation and deleteAnnotation methods, as they are no longer used. - Factor out everything but the event publication of deleteAnnotation into a detach method that is the inverse of the anchor method. - Subscribe to annotationsLoaded and annotationDeleted and invoke anchor and detach accordingly.
-
- 12 Aug, 2015 5 commits
-
-
Randall Leeds authored
Like all the other calls, this should invoke its callback.
-
Randall Leeds authored
This ensures there's a digest cycle on the angular side after the client state is synchronized and brings this into symmetry with the other RPC calls defined by AnnotationSync.
-
Randall Leeds authored
This causes unnecessary network traffic and momentarily blanks the ephemeral properties of the annotation set on the client.
-
Randall Leeds authored
Make setupAnnotation into a tiny compatibility wrapper around a new method #anchor() method that returns a promise of the anchors. This is cleaner than trying to store the anchors on the annotation itself, which creates circular references, and it means there's no need to strip these properties when serializes annotations across the frame boundary. Sort annotations by TextPositionSelector in the widget display. This provides greater visual stability -- cards don't change positions after anchoring -- at the cost of potentially out of order cards when anchors have moved.
-
Randall Leeds authored
We're not displaying time in any timezones other than the browser local one. Therefore, we don't need to parse and convert time zone names. Moment will pull the timezone offset itself from the current browser by using Date.prototype.getTimezoneOffset(). This eliminates 251kB (43kB gzipped) of minified code from our application bundle. Wow!
-