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

New rendering: Move the article lead image #4398

Open
romu70 opened this issue May 15, 2020 · 0 comments
Open

New rendering: Move the article lead image #4398

romu70 opened this issue May 15, 2020 · 0 comments
Labels

Comments

@romu70
Copy link

romu70 commented May 15, 2020

⚠️ If your issue is about an error during fetching a link, please read: http://doc.wallabag.org/en/user/errors_during_fetching.html#how-can-i-help-to-fix-that

Issue details

New rendering improvement.

When the article has a lead image, it should be placed in the first place, so before the H1. In my testing, I took the "header" section which contains the lead image and moved to between the div with the "article" ID and the header with the "mbm" class.

But feel free to do it differently ;)

Environment

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

Steps to reproduce/test case

Please provide necessary steps for reproduction of this issue, or better the
reduced test case (without any external dependencies, if possible).

@romu70 romu70 changed the title Move the article lead image New rendering: Move the article lead image May 16, 2020
@Kdecherf Kdecherf added the UI label Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants