Skip to content
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

9 header component [WIP] #116

Closed
wants to merge 2 commits into from

Conversation

kaczynska
Copy link

No description provided.

.v-menu {
margin-right: 20px;
}
</style>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No newline

@w1stler
Copy link
Member

w1stler commented Aug 21, 2019

Do we want to have logo in header instead of plain text? Currently it's this way. We should also remove the white box beneath the header.

https://ibb.co/0265wWP
https://ibb.co/RCxjcY1

@kaczynska kaczynska changed the title 9 header component 9 header component [WIP] Sep 18, 2019
@kaczynska
Copy link
Author

Do we want to have logo in header instead of plain text? Currently it's this way. We should also remove the white box beneath the header.

https://ibb.co/0265wWP
https://ibb.co/RCxjcY1

We want to use the svg format because it is optimal and completely compatible with the original version of the character. When using text, we do not have control over the distance between letters, and these in practice are often adjusted manually (at the stage of working on the sign).

The white box should of course be removed.

@@ -0,0 +1,48 @@
<?xml version="1.0" encoding="utf-8"?>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if we're going to have logos in SVG can we also store them in https://github.com/CodeForPoznan/Community/tree/master/logos

@w1stler w1stler closed this Jan 8, 2020
@w1stler
Copy link
Member

w1stler commented Jan 8, 2020

Closing - @stanislawK is working on this in #227

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants