Skip to content

feat(link): Новые стили в links (#183685) #127

Merged
merged 5 commits into from
Jun 6, 2019
Merged

feat(link): Новые стили в links (#183685) #127

merged 5 commits into from
Jun 6, 2019

Conversation

lskramarov
Copy link
Contributor

No description provided.

@lskramarov lskramarov requested a review from mikeozornin May 27, 2019 07:00
@lskramarov lskramarov self-assigned this May 27, 2019
@mikeozornin
Copy link
Contributor

@fost, там что-то с многострочными ссылками, @lskramarov посмотрит.

@lskramarov
Copy link
Contributor Author

Нужно поправить многострочность

mikeozornin
mikeozornin suggested changes Jun 5, 2019
View reviewed changes
Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

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

Многострочность норм.

У меня ссылки выходят за границы блока: http://d.mikeozornin.ru/gpv6AN. Это из-за паддинга, не знаю зачем он. Из-за этого ссылки будут часто не так выравниваться по сравнению с соседними элементами.

@lskramarov
Copy link
Contributor Author

Многострочность норм.

У меня ссылки выходят за границы блока: http://d.mikeozornin.ru/gpv6AN. Это из-за паддинга, не знаю зачем он. Из-за этого ссылки будут часто не так выравниваться по сравнению с соседними элементами.

паддинги были изначально, пока непонятно зачем они, возможно безболезненно можно убрать, нужно смотреть.

@lskramarov
Copy link
Contributor Author

паддинги для рамки фокуса + вылезла проблема с ним при многострочности
image

@lskramarov
Copy link
Contributor Author

image

думаю можно вливать

@pimenovoleg pimenovoleg self-requested a review June 6, 2019 08:39
@mikeozornin mikeozornin self-requested a review June 6, 2019 14:22
mikeozornin
mikeozornin approved these changes Jun 6, 2019
View reviewed changes
Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

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

Я сам не смотрел, но на последнем скриншоте все отлично. Так можно вливать.

@pimenovoleg pimenovoleg merged commit 9df97f7 into master Jun 6, 2019
pimenovoleg pushed a commit that referenced this pull request Jun 7, 2019
@lskramarov @pimenovoleg
feat(link): Новые стили в links (#183685) (#127)
31d6b87
pimenovoleg pushed a commit that referenced this pull request Jun 7, 2019
@lskramarov @pimenovoleg
feat(link): Новые стили в links (#183685) (#127)
3762afc
@pimenovoleg pimenovoleg deleted the feat/183685-new-styles0in-links branch June 8, 2019 09:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mikeozornin mikeozornin mikeozornin approved these changes

@pimenovoleg pimenovoleg pimenovoleg approved these changes

Assignees

@lskramarov lskramarov

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lskramarov @mikeozornin @pimenovoleg