New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LWN.net articles get cut off #2825

Closed
anarcat opened this Issue Feb 1, 2017 · 9 comments

Comments

Projects
None yet
3 participants
@anarcat

anarcat commented Feb 1, 2017

Issue details

This email on LWN.net is actually quite readable on the fivefilters test site. But in wallabag, it gets cut off on the side, probably because of some CSS issue.

Here's a screenshot showing the problem:

LWN.net with text cut off

It makes the whole article basically unreadable. This is especially bad on the EPUB output...

Environment

  • wallabag version (or git revision) that exhibits the issue: 2.1.5
  • How did you install wallabag? Via git clone or by downloading the package? unknown
  • Last wallabag version that did not exhibit the issue (if applicable): unknown
  • php version: unknown
  • OS: Debian GNU/Linux 8 "jessie"
  • type of hosting (shared or dedicated): dedicated
  • which storage system you choose at install (SQLite, MySQL/MariaDB or PostgreSQL): unknonwn

Steps to reproduce/test case

Add this article to wallbag, try to read it. Try it on the fivefilters test site, and actually read it correctly.

@j0k3r

This comment has been minimized.

Show comment
Hide comment
@j0k3r

j0k3r Feb 2, 2017

Member

Looks ok to me on app.wallabag.it:

image

Can you share your browser and your resolution?
Maybe you should upgrade to 2.2.1.

Member

j0k3r commented Feb 2, 2017

Looks ok to me on app.wallabag.it:

image

Can you share your browser and your resolution?
Maybe you should upgrade to 2.2.1.

@anarcat

This comment has been minimized.

Show comment
Hide comment
@anarcat

anarcat Feb 2, 2017

the web browser here is Firefox 50.1.0 (Build identifier: Mozilla/5.0 (X11; Linux x86_64; rv:50.0) Gecko/20100101 Firefox/50.0) running on Debian GNU/Linux 9 "stretch". Screen resolution is:

[996]anarcat@curie:~$ xrandr -q
Screen 0: minimum 320 x 200, current 1680 x 1050, maximum 8192 x 8192
HDMI-1 disconnected primary (normal left inverted right x axis y axis)
DP-1 connected 1680x1050+0+0 (normal left inverted right x axis y axis) 473mm x 296mm
   1680x1050     59.95*+

The font picked may be the problem: here it is DejaVu Sans Mono - maybe it's wider than your font?

anarcat commented Feb 2, 2017

the web browser here is Firefox 50.1.0 (Build identifier: Mozilla/5.0 (X11; Linux x86_64; rv:50.0) Gecko/20100101 Firefox/50.0) running on Debian GNU/Linux 9 "stretch". Screen resolution is:

[996]anarcat@curie:~$ xrandr -q
Screen 0: minimum 320 x 200, current 1680 x 1050, maximum 8192 x 8192
HDMI-1 disconnected primary (normal left inverted right x axis y axis)
DP-1 connected 1680x1050+0+0 (normal left inverted right x axis y axis) 473mm x 296mm
   1680x1050     59.95*+

The font picked may be the problem: here it is DejaVu Sans Mono - maybe it's wider than your font?

@j0k3r

This comment has been minimized.

Show comment
Hide comment
@j0k3r

j0k3r Feb 2, 2017

Member

I've just tested on the same Firefox version on Ubuntu and still not get the problem.
The font is monospace here.

Member

j0k3r commented Feb 2, 2017

I've just tested on the same Firefox version on Ubuntu and still not get the problem.
The font is monospace here.

@anarcat

This comment has been minimized.

Show comment
Hide comment
@anarcat

anarcat Feb 2, 2017

yeah, monospace is DejaVu Sans Mono here. not sure what it is for you - in the stylesheet, monospace is just an alias for the system monospace font: you need to go to the "Fonts" tab to figure out exactly what font is used...

anarcat commented Feb 2, 2017

yeah, monospace is DejaVu Sans Mono here. not sure what it is for you - in the stylesheet, monospace is just an alias for the system monospace font: you need to go to the "Fonts" tab to figure out exactly what font is used...

@j0k3r

This comment has been minimized.

Show comment
Hide comment
@j0k3r

j0k3r Feb 2, 2017

Member

Interesting.
So the fond used is also DejaVu Sans Mono here.

I guess the problem have been fixed in 2.2

Member

j0k3r commented Feb 2, 2017

Interesting.
So the fond used is also DejaVu Sans Mono here.

I guess the problem have been fixed in 2.2

@nicosomb

This comment has been minimized.

Show comment
Hide comment
@nicosomb

nicosomb Feb 4, 2017

Member

@anarcat can you tell us if it looks ok for you with latest release?

Member

nicosomb commented Feb 4, 2017

@anarcat can you tell us if it looks ok for you with latest release?

@anarcat

This comment has been minimized.

Show comment
Hide comment
@anarcat

anarcat Feb 4, 2017

where can i test? my hoster didn't upgrade yet...

anarcat commented Feb 4, 2017

where can i test? my hoster didn't upgrade yet...

@nicosomb

This comment has been minimized.

Show comment
Hide comment
@nicosomb

nicosomb Feb 5, 2017

Member

you can create a trial account on wallabag.it.

Member

nicosomb commented Feb 5, 2017

you can create a trial account on wallabag.it.

@anarcat

This comment has been minimized.

Show comment
Hide comment
@anarcat

anarcat Feb 6, 2017

sure looks like the font (size) is different, i guess this is fixed.

anarcat commented Feb 6, 2017

sure looks like the font (size) is different, i guess this is fixed.

@anarcat anarcat closed this Feb 6, 2017

@j0k3r j0k3r removed the Awaiting response label Feb 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment