Skip to content

Обновляет лого для JS #53

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

Merged
merged 6 commits into from
Oct 27, 2021
Merged

Conversation

nikolai-shabalin
Copy link
Contributor

No description provided.

Copy link
Contributor

@sashasushko sashasushko left a comment

Choose a reason for hiding this comment

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

Кажется, что лого другое должно быть - без цифры

@nikolai-shabalin
Copy link
Contributor Author

Это не цифра, это буква S. Тот логотип с JS имеет другие пропорции. То есть чтобы его использовать нужно будет и HTML менять

@nikolai-shabalin
Copy link
Contributor Author

Это всё из-за надписи над логотипом html =(

@sashasushko
Copy link
Contributor

Так может бахнем сразу троицу, как по моей ссылке? HTML и CSS пусть ведут на вашу часть, JS - на нашу

@nikolai-shabalin
Copy link
Contributor Author

Давай так

@nikolai-shabalin
Copy link
Contributor Author

@sashasushko обновил

@sashasushko
Copy link
Contributor

Всё забываю ответить: давайте JS-лого оставим квадратным, а CSS и HTML сопоставимые по высоте, и ничего что они тогда будут шире.

@sashasushko
Copy link
Contributor

Закинул коммит, чтобы было понятно, о чём я.

# Conflicts:
#	css/style.css
@nikolai-shabalin
Copy link
Contributor Author

Ага, подмёржил мастер. Конфликты были. Давайте мёржить?

@nikolai-shabalin nikolai-shabalin merged commit 066499f into master Oct 27, 2021
@nikolai-shabalin nikolai-shabalin deleted the fix/new-js-logo branch October 27, 2021 15:03
nikolai-shabalin added a commit that referenced this pull request Oct 27, 2021
* Обновляет лого для JS

* Удаляет лишние атрибуты

* Делает три логотипа: HTML, CSS, JS

* Сопоставит размеры JS-лого с остальными

(cherry picked from commit 066499f)
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