Skip to content
This repository has been archived by the owner on Apr 16, 2023. It is now read-only.

Feature/rework landing page emblems #772

Merged
merged 12 commits into from
Dec 13, 2016

Conversation

das-g
Copy link
Member

@das-g das-g commented Dec 13, 2016

before:

screenshot of frontpage before change

after:

screenshot of frontpage after change

@sfkeller
Copy link

The new emblems graphics is OK for me.

I just see that the buttons still don't have the same text as the menu. I'd prefer a hover text over the two emblems, and that the emblems are "clickable".

Even more, I'm wondering, if the buttons below the two emblems can't be completely omitted?

@das-g
Copy link
Member Author

das-g commented Dec 13, 2016

I just see that the buttons still don't have the same text as the menu. I'd prefer a hover text over the two emblems, [...]

Good point. Let's handle that in a separate pull request, though: #773 (hovertext only over buttons, but we I can change that)

I'd prefer a hover text over the two emblems, and that the emblems are "clickable".

Even more, I'm wondering, if the buttons below the two emblems can't be completely omitted?

Maybe the caption and the emblems should be together on a button, to indicate that the whole thing is clickable?

@sfkeller sfkeller merged commit 52d873a into develop Dec 13, 2016
@das-g das-g deleted the feature/rework_landing_page_emblems branch December 14, 2016 08:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants