Hi!

sajolida:
> u:
>> for the future donation campaign I need a clean start..
>> That's why I implemented some small CSS fixes. Please review and merge:
>>
>> 451f:tails/feature/smallCSSfixes
> 
> Hey!
> 
> It's so cool that you are doing some housekeeping on our CSS.
> It definitely needs it!
> 
> Some comments:
> 
> - I'm not a fan of the mobile version of the logo. I understand that you
> put the subtitle "the amnesic incognito live system" in a smaller font
> so that it fits on the screen but now the font is barely readable for me
> (with my glasses on and no colorblindness). I wonder if we should
> instead get rid of the subtitle in the mobile version

> And I came to wonder whether we should even get rid of it in all
> versions of the logo or make it optional as it's not very appropriate
> for smaller rendering like in the banner or on small stickers.
> Removing it might help us solve what I see as a uneasy tension in the
> vertical alignment between (Tails + the amnesic incognito live system)
> and the icon.

I simply want the logo to fit on a small screen, yes.
I think we can indeed get rid of the font on the small version.

> Also, in your mobile version you pretty much reuse the font size of the
> logo where "the amnesic incognito live system" is as wide as "Tails" but
> you also added some extra vertical space that is not present neither in
> the logo nor on the desktop banner. Is there a reason for that?

I did not add any extra vertical space as far as I know. So I don't know
where you see that?

> As a more general note, in terms of style guide (I agree that so far we
> have none), if we want to spend time on the display of our logo, I'd
> rather go in the direction of using our logo in more similar ways
> everywhere than creation new subtle variations.

Agreed.

> But I reckon that we're doing this already in the banner so I can't
> really blame you for continuing doing so :) And I'm not at all saying
> that you should go edit our logo as part of this but maybe having some
> discussion about it could be a first step towards and new a better version.

Yes, I figured that the current banner is actually not like what exists
on /promote... And I wondered how that version came to life.

So I'm fine with keeping the banner as is, and for the mobile version
just remove the text below.

> - I like the fixes on the sidebar and three-blocks for mobiles!
> 
> - I like the fixes to the breadcrumbs!
> 
> So after you gave me your take on the logo issue we're ready to merge!

Cool!

u.
_______________________________________________
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.

Reply via email to