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
Added creation date and reading time on article view #2222
Conversation
{% set readingTime = entry.readingTime / app.user.config.readingSpeed %} | ||
<div class="link mdi-action-query-builder"> {% if readingTime > 0 %}{{ 'entry.list.reading_time_minutes_short'|trans({'%readingTime%': readingTime|round}) }} | ||
{% else %}{{ 'entry.list.reading_time_less_one_minute_short'|trans|raw }} | ||
{% endif %}</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't this be displayed more nicely? I mean, it's hard to read the if / else / endif with that kind of indentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
Seems good. Am I the only one that the font kind of disturbs ? Or is it the margin around the icons ? |
7475777
to
e7732de
Compare
{% else %} | ||
{{ 'entry.list.reading_time_less_one_minute_short'|trans|raw }} | ||
{% endif %} | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of using icons (which aren't available) display real text for reading time & creation date
@tcitworld yeah but I don't know how to fix it.. |
👍 |
Fix #2034 (from @foxmask)