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

Private links counter in the header #560

Merged
merged 2 commits into from
May 14, 2016

Conversation

ArthurHoaro
Copy link
Member

Also moved PageBuilder to its own file to clean up index.php.

Fixes #247

@ArthurHoaro ArthurHoaro self-assigned this May 10, 2016
@ArthurHoaro ArthurHoaro added enhancement cleanup code cleanup and refactoring template HTML rendering in review labels May 10, 2016
@ArthurHoaro ArthurHoaro added this to the 0.7.0 milestone May 10, 2016
@@ -2,7 +2,8 @@
<div id="logo" title="Share your links !" onclick="document.location='?';"></div>

<div id="linkcount" class="nomobile">
{if="!empty($linkcount)"}{$linkcount} links{/if}
{if="!empty($linkcount)"}{$linkcount} links{/if}<br>
{if="!empty($privateLinkcount)"}{$privateLinkcount} privates{/if}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/privates/private links/

this is a subtle one for non-English speakers, as "privates" is a way of designating one's genitalia - http://www.freethesaurus.com/privates

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't it exactly why one would save links in private mode? :)

@ArthurHoaro ArthurHoaro merged commit 3fdcc7b into shaarli:master May 14, 2016
@ArthurHoaro ArthurHoaro deleted the nb-private-shaare branch May 14, 2016 09:11
@virtualtam virtualtam mentioned this pull request May 17, 2016
43 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup code cleanup and refactoring enhancement in review template HTML rendering
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants