Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Add word wrap of text in the Outline view to avoid horizontal scrolling #2510

Merged
merged 1 commit into from

3 participants

@Snuffleupagus
Collaborator

This patch attempts to reduce/remove the need for horizontal scrolling of the outline view by wrapping long lines.
In my testing it works really well, but it might cause problems in situations that I cannot test. (For instance I don't know if this works in RTL languages.)

@yurydelendik

/botio-windows preview

@pdfjsbot
Collaborator

From: Bot.io (Windows)


Received

Command cmd_preview from @yurydelendik received. Current queue size: 2

Live output at: http://107.22.172.223:8877/a85a203d1ca57c4/output.txt

@yurydelendik

Yes, it does not work for RTL. To test specify the locale in the parameters, e.g. http://107.22.172.223:8877/a85a203d1ca57c4/web/viewer.html#locale=ar (if you have time, look into that later)

Looks good. Thank you for the patch.

@yurydelendik yurydelendik merged commit 956d5b0 into from
@Snuffleupagus Snuffleupagus deleted the branch
@Snuffleupagus Snuffleupagus referenced this pull request from a commit in Snuffleupagus/pdf.js
Snuffleupagus Fixes the Outline view in RTL languages, follow up of #2510 d3836c0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 27, 2012
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 3 deletions.
  1. +3 −3 web/viewer.css
View
6 web/viewer.css
@@ -975,8 +975,8 @@ a:focus > .thumbnail > .thumbnailSelectionRing,
text-decoration: none;
display: inline-block;
min-width: 95%;
- height: 20px;
- padding: 2px 0 0 10px;
+ height: auto;
+ padding: 2px 0 5px 10px;
margin-bottom: 1px;
border-radius: 2px;
color: hsla(0,0%,100%,.8);
@@ -984,7 +984,7 @@ a:focus > .thumbnail > .thumbnailSelectionRing,
line-height: 15px;
-moz-user-select:none;
cursor: default;
- white-space: nowrap;
+ white-space: normal;
}
.outlineItem > a:hover {
Something went wrong with that request. Please try again.