Skip to content

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

Merged
nikolai-shabalin merged 6 commits into
masterfrom
fix/new-js-logo
Oct 27, 2021
Merged

nikolai-shabalin merged 6 commits into
masterfrom
fix/new-js-logo

Conversation

@nikolai-shabalin

Copy link
Copy Markdown
Contributor

No description provided.

@sashasushko sashasushko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

@nikolai-shabalin

Copy link
Copy Markdown
Contributor Author

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

@nikolai-shabalin

Copy link
Copy Markdown
Contributor Author

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

@sashasushko

Copy link
Copy Markdown
Contributor

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

@nikolai-shabalin

Copy link
Copy Markdown
Contributor Author

Давай так

@nikolai-shabalin

Copy link
Copy Markdown
Contributor Author

@sashasushko обновил

Comment thread index.html Outdated
@sashasushko

Copy link
Copy Markdown
Contributor

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

@sashasushko

Copy link
Copy Markdown
Contributor

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

# Conflicts:
#	css/style.css
@nikolai-shabalin

Copy link
Copy Markdown
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