-
Notifications
You must be signed in to change notification settings - Fork 15
Update member logos #545
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
base: master
Are you sure you want to change the base?
Update member logos #545
Conversation
Signed-off-by: [email protected] <[email protected]>
Signed-off-by: [email protected] <[email protected]>
|
Preview your changes here |
june-andronick
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
Signed-off-by: [email protected] <[email protected]>
|
Preview your changes here |
lsf37
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should do this (I mean the text/SVG part, the add/remove is fine of course). Logos should be logos and not text. If a member does not want to show their logo, we should make an SVG that shows text so the code remains uniform.
Also, why are we changing TUM?
|
the Apple logo is too big, it dominates the whole page. |
As members sign the new agreements, they can pick again whether they allow the use of their logo. In the case of TUM, we were told that the university has become more restrictive, hence no logo. |
This must be just the preview. It looks fine on the live page and that part hasn't changed (RTX also looks too big on the preview)
UNSW appears as founding member and service provider in the preview for me. The other founding members (aside DW and UNSW) are now gone (or merged into another entity) |
|
Ahh, I got confused. Because the "Founding Member" and "service provider" annotations are so far from the logo, they look like a page footer rather than an annotation of the member logo. Quite spaced at some of the other members too (eg Kry10) but it's less confusing as they aren't at the bottom of the page |
It's not the preview, I can see the same problem when I build it locally. It might be the code change that I want to revert, though. This might also explain the broken spacing of the founding/service provider lines, although I suspect that in this case it is the fault for the trusted computing logo being too large -- the bylines need to be aligned underneath the logos, which means if one of the logos sticks out, all of the bylines need to increase spacing.
As an alumni of TUM, let me just say 🙄. It's fine to have the logo on the age old coffee cup in a public cupboard, but it is not fine to have it on the website of the org they just joined. Looks like stupid in admin is winning globally. For these cases, we should still make a logo file with text, though. Trying to render text in the browser predictably is a road to madness and failure. |
Signed-off-by: [email protected] <[email protected]>
|
Preview your changes here |
|
SVGs for text logos created. NB All text logos need to be created using the same parameters, otherwise font sizes will be different post rendering. Instructions are in the readme. |
of course, we could bottom-align the logos |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you mean to commit this file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. Riverside Research also don't want their logo displayed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But this is not the Riverside Research file (there is a separate one for that).
| Text logos are used in cases where associations haven't given permission for their logo to be displayed on the website. For example, TU Munich. | ||
|
|
||
| Text logos (SVG) are generated using Inkscape with the following settings: | ||
| - Font: Roboto Mono Nerd Font |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like this doesn't quite work -- you either need to use a font that is guaranteed to exist in the browser (e.g. a font the website already uses like plain Roboto) or convert it to outlines so that what you see will be displayed. E.g. my browser shows me Times New Roman.
|
Gernot writes:
That only fixes this one immediate issue, not the overall layout of logos in different contexts. What we mostly want is a similar aspect ratio, and depending on situation you want to either constrain height or width. That is all possible (and looks fine on the current site) if they are pictures, and gets hard if some are pictures and some are text. Birg writes:
We don't actually need the font size to be the same, we just need the aspect ratios to be similar enough. It's perfectly fine if e.g. the font for the short "TU Munich" is larger than for the long "Riverside Research Institute". What it mainly needs is the layout like you have done it. We do want the font face to be the same, though. (ps: pretty sure "TU Munich" is verboten according to their own guidelines, it must most likely be "TU München" or "TUM", but I enjoy the irony so I would leave it as is :-)). |
No description provided.