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

Added tum bar at top #491

Closed
wants to merge 7 commits into from
Closed

Added tum bar at top #491

wants to merge 7 commits into from

Conversation

yashha
Copy link
Collaborator

@yashha yashha commented Apr 22, 2016

image

@mammuth
Copy link
Collaborator

mammuth commented Apr 22, 2016

You might want to put some padding left of the technische Universität München and around the tum logo.

@yashha
Copy link
Collaborator Author

yashha commented Apr 22, 2016

What about putting it ontop of the image/nav?

@yashha
Copy link
Collaborator Author

yashha commented Apr 22, 2016

image

on hover:
image

@mammuth
Copy link
Collaborator

mammuth commented Apr 22, 2016

I'd prefer the grey version. Because it looks like the bar is not part of the actual page...

@yashha
Copy link
Collaborator Author

yashha commented Apr 22, 2016

like this?
image

@mammuth
Copy link
Collaborator

mammuth commented Apr 22, 2016

Like your first version
0e1b4578-08ce-11e6-8c20-5fe2cfd422d0
This ugly bar is no part of the website. In this version it's visually no part as well.

@yashha
Copy link
Collaborator Author

yashha commented Apr 22, 2016

I don't like that it is that white.
I would like the second approach.
What does the other guys say?
@jorjo1 @gyachdav @sacdallago

@gyachdav
Copy link
Collaborator

I'd go for this

image

Without the hover (the hover is not a requirement is it?)

This is the least obtrusive version

Have some margins/padding around the text so it wont be stuck to the borders like it is now.

@yashha
Copy link
Collaborator Author

yashha commented Apr 22, 2016

image

@gyachdav
Copy link
Collaborator

@yashha can you move the text closed to the left border and the logo to the right border?

@yashha
Copy link
Collaborator Author

yashha commented Apr 22, 2016

Its possible I think, but not easy.

@mammuth
Copy link
Collaborator

mammuth commented Apr 22, 2016

Why not just make it as it was before and add a padding left/right?

On Fri, Apr 22, 2016, 22:57 yashha notifications@github.com wrote:

Its possible I think, but not easy.


You are receiving this because you were assigned.

Reply to this email directly or view it on GitHub
#491 (comment)

@yashha
Copy link
Collaborator Author

yashha commented Apr 22, 2016

I think it doesn't look that good too.

image

Maybe it looks better with fullsize background image

@jorjo1
Copy link
Contributor

jorjo1 commented Apr 22, 2016

@yashha I fixed the image, it has now full size, just merge my branch..

@sacdallago
Copy link
Contributor

sacdallago commented Apr 22, 2016

Guys. Guys. Slow down! As I told you there are specific rules on colors, position and other things!

Please:

  • No hover,
  • try to use the same font as on the tum webpage (I think they have a PNG also for the FQDN 😂 Techinsche Universität München ),
  • make the whole thing a thick bigger (otherwise it looks like a mistake we forgot there),
  • there should be padding above and under the tum logo and tum name, same as in comment Legal mattes #482 (comment) (try to stick to that color scheme (which should fit the got.show style) and general formatting as much as possible)

I suggest you to not have the logo right:0 and the FDQN left:0, but to have a div of the same size of the page container (ie. width == to header width, 550 on my screen but it surely depends on screen size) and then inside this centered div have the two logos on right:0 and left:0

I hate to be the one making you do this, but implementing it to then have the legal office complain about it again is also not gonna bring us any further!

Again: Thank you sooooo much for doing this!

EDIT: Here you have the color-scheme and some other hints: https://portal.mytum.de/pressestelle/tum_mit/2007nr2/09.pdf

@yashha yashha closed this Apr 23, 2016
@yashha yashha deleted the yashha-fix-6 branch June 5, 2016 22:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

5 participants