<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Apr 5, 2018, 22:56 Dominik Ruf <<a href="mailto:dominikruf@gmail.com">dominikruf@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">Dominik Ruf <<a href="mailto:dominikruf@gmail.com" target="_blank" rel="noreferrer">dominikruf@gmail.com</a>> schrieb am Di., 27. März 2018 um 23:17 Uhr:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Mar 27, 2018, 21:07 Thomas De Schampheleire <<a href="mailto:patrickdepinguin@gmail.com" target="_blank" rel="noreferrer">patrickdepinguin@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2018-03-26 19:07 GMT+02:00 Dominik Ruf <<a href="mailto:dominikruf@gmail.com" target="_blank" rel="noreferrer">dominikruf@gmail.com</a>>:<br>
> Hi again,<br>
><br>
> and thanks again for the feedback last time.<br>
> I made some progress and would love to hear what you think about it.<br>
> <a href="https://kallithea-exp.dominikruf.com/kallithea/kallithea" rel="noreferrer noreferrer" target="_blank">https://kallithea-exp.dominikruf.com/kallithea/kallithea</a><br>
> <a href="https://kallithea-exp.dominikruf.com/kallithea/kallithea-domruf/changelog" rel="noreferrer noreferrer" target="_blank">https://kallithea-exp.dominikruf.com/kallithea/kallithea-domruf/changelog</a><br>
><br>
<br>
On mobile phone, the sidebar is hidden by default (which is fine).<br>
Clicking the menu button on the second navbar makes it visible (also<br>
fine) but then: when you click the menu button again, there is an<br>
animation of the sidebar 'rolling up', and it takes too long IMO. It<br>
is not as quick as the expansion, and makes it feel sluggish.<br></blockquote></div><div>This is the default animation of bootstrap. I thought about that myself but it didn't bother me enough to change it yet :-D</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Other than that,  it looks nice, both on mobile as on desktop. (on<br>
mobile there are some general non-ideal points but not specific to the<br>
sidebar).<br>
<br>
I like the 'number of pullrequests' overlay on the PR button, but I do<br>
wonder how it will look like if the number is 2 or even 3 digits long.<br></blockquote></div><div>It think it looks good with 2 digits and if you have more then 100 open pull requests I believe there is something wrong with your project. :-D</div><div>BTW you can see how it looks by simply changing the text with the debugger of your browser.</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The '@' sign to indicate the revision in plain text seems a bit<br>
unprofessional. I think it could be improved in some way, but not sure<br>
how. Perhaps with an 'at' icon.<br></blockquote></div><div>I don't like the font for the revision hash (and @). I'm not sure what you mean by an 'at' icon though.</div></blockquote><div>Can you tell me a bit more what you mean by "an 'at' icon" ? </div><div>BTW I worked a bit more on it and now all the pages use the new look. (I hope I didn't forget one) And the animation should be fix now.</div><div><a href="https://kallithea-exp.dominikruf.com" target="_blank" rel="noreferrer">https://kallithea-exp.dominikruf.com</a><br></div><div></div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Sorry I forgot to reply on the at. What I meant was to use an at symbol from fontello. I searched there and found several at symbols which are a bit bolder than the normal bold text. I think that looks nicer.</div></div>