search

documentcloud / document-viewer

The NYTimes Document Viewer
http://open.blogs.nytimes.com/2010/03/27/a-new-view-introducing-doc-viewer-2-0/
Apache License 2.0
690 stars 119 forks source link

Suggestion: Remove annotation padding #40

Open roberttdev opened 10 years ago

roberttdev commented 10 years ago

I've been using DocumentCloud to highlight scientific documents, and we were struggling a bit when it came to highlighting multiple cells in a small table (even with zoom). There were 'dead zones' where the highlight cursor changed to the page grab cursor, near an existing annotation and in a small page-spanning row above it.

I was able to remove this issue by removing the 'padding: 3px' property from '.DV-annotation'. To help users trying to make similarly close highlights, this may be helpful to change.

knowtheory commented 10 years ago

Any interest in tossing us a pull request, so you get credit for the change? :)

roberttdev commented 10 years ago

Certainly! #41