Skip to content
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

justification #554

Closed
Kaligule opened this issue Mar 13, 2014 · 11 comments
Closed

justification #554

Kaligule opened this issue Mar 13, 2014 · 11 comments
Milestone

Comments

@Kaligule
Copy link
Contributor

Text would be far more readable if there was some justification.

@nicosomb
Copy link
Member

I think justification on the web is not ... justified, contrary to newspaper for example.
We can find many articles about this topic on Google (https://www.google.fr/#q=web+justification)

I don't think justification is a good thing.

I leave this issue opened.

@Kaligule
Copy link
Contributor Author

You might be right (although I don't agree), but with Version 2.0 wallabag is going to be on all kind of devices.

  • eReaders
  • Smartphones with small displays (iPhones, even Featurephones)
  • text crawling into space like in Starwars

At least for two of those I can asure you that unjustificated text in long articles is really hard to read. (I try every day in the wallabag app, but mostly have to switch back to Pocket with it) I really think justification is a good thing.

So what about a justification switch in the settings?

On 13.03.2014, at 13:57, Nicolas Lœuillet notifications@github.com wrote:

I think justification on the web is not ... justified, contrary to newspaper for example.
We can find many articles about this topic on Google (https://www.google.fr/#q=web+justification)

I don't think justification is a good thing.

I leave this issue opened.


Reply to this email directly or view it on GitHub.

@nicosomb nicosomb added this to the 2.0 milestone Mar 13, 2014
@nicosomb
Copy link
Member

A setting can be a good thing. I add this ticket to v2 roadmap.

@Kaligule
Copy link
Contributor Author

Thank you :)

On 13.03.2014, at 14:45, Nicolas Lœuillet notifications@github.com wrote:

A setting can be a good thing. I add this ticket to v2 roadmap.


Reply to this email directly or view it on GitHub.

@mariroz
Copy link
Contributor

mariroz commented Mar 13, 2014

hm,
and what is the problem to set style "text-align: justify;" for an article?
this can be done in a second, if you find it is needed.

@Kaligule
Copy link
Contributor Author

You can do that and I probably could find out how it is done (although I am not good with HTML and PHP). But in said usecases this might not be possible.

On 13 Mar 2014, at 15:44, mariroz notifications@github.com wrote:

hm,
and what is the problem to set style "text-align: justify;" for an article?
this can be done in a second, if you find it is needed.


Reply to this email directly or view it on GitHub.

@NumEricR
Copy link
Contributor

I think users should be able to choose font-family, default size and justification in config panel.

@tcitworld
Copy link
Member

I agree with font-family, but not with default size, because this should be a browser setting, not a webpage setting.

(OK, that's @nicosomb's argument.)

@nicosomb
Copy link
Member

yes, totally agree with @tcitworld :)

nicosomb added a commit that referenced this issue Mar 13, 2014
…issue-505

saving link form now sends ajax request, article view is now justified (issue #554)
@mariroz
Copy link
Contributor

mariroz commented Mar 13, 2014

@Kaligule, justification is now temporary added in dev branch (baggy theme), pls try.
even if we will remove it, you can always add it to your copy of wallabag: just one line in css file.

@Kaligule
Copy link
Contributor Author

Thank you, I definitely will do that

On 13.03.2014, at 22:43, mariroz notifications@github.com wrote:

@Kaligule, justification is now temporary added in dev branch (baggy theme), pls try.
even if we will remove it, you can always add it to your copy of wallabag: just one line in css file.


Reply to this email directly or view it on GitHub.

@nicosomb nicosomb modified the milestones: 2.0, 2.0.0-alpha Aug 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants