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

Добавил новые компании из списка #229

Merged
merged 2 commits into from
Aug 12, 2021

Conversation

azinit
Copy link
Member

@azinit azinit commented Aug 12, 2021

CHANGELOG

  • Добавил новые компании из списка

Чеклист

  • Если при работе с документацией потребовалось использовать github-дискуссии, то стоит их прикрепить как see-also источники
  • Если PR связан с задачей, то необходимо проверить, что все требования по задаче выполнены
  • Перед тем, как отправлять изменения на ревью, нужно провести self-review своих изменений
  • Перед тем, как отправлять изменения на ревью, нужно дождаться CI-проверок
  • Перед тем, как отправлять изменения на ревью, нужно дать краткое описание изменений

@GhostMayor
Copy link

может почистить svg от id="Layer_1" data-name="Layer 1"

Postamentovich
Postamentovich previously approved these changes Aug 12, 2021
@azinit
Copy link
Member Author

azinit commented Aug 12, 2021

может почистить svg от id="Layer_1" data-name="Layer 1"

Сразу видно солдат мегаполиса)))

Тоже были мысли такие, ну давай

@azinit
Copy link
Member Author

azinit commented Aug 12, 2021

@GhostMayor
Copy link

Вот это как может быть в иконке ?

transform="translate(-2.43 -3.03)"

какой трансформ ))

@azinit
Copy link
Member Author

azinit commented Aug 12, 2021

Вот это как может быть в иконке ?

Я хз)

Но это иконку такую для компании скинули, так что некритично думаю

UPD: Еще и не такого навидаемся

Copy link

@GhostMayor GhostMayor left a comment

Choose a reason for hiding this comment

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

Осуждаю такие svg )

@azinit
Copy link
Member Author

azinit commented Aug 12, 2021

Осуждаю такие svg )

Осуждаем, не одобряем)))

@azinit azinit merged commit d088144 into master Aug 12, 2021
@azinit azinit deleted the feature/add-companies-2 branch August 12, 2021 19:18
Copy link
Member

@Postamentovich Postamentovich left a comment

Choose a reason for hiding this comment

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

Иконку нужно пожать, например с помощью https://jakearchibald.github.io/svgomg/

@Postamentovich
Copy link
Member

Эх

@azinit
Copy link
Member Author

azinit commented Aug 12, 2021

@Postamentovich @GhostMayor мы всегда будем рады вашим PRам по оптимизации иконок 😎

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.

4 participants