Skip to content

refactor(amount): make amount markup flat #1105

Merged
merged 2 commits into from Apr 17, 2020

Conversation

reme3d2y
Copy link
Contributor

@reme3d2y reme3d2y commented Mar 12, 2020

Мотивация и контекст

Упрощена разметка компонента Amount в рамках задачи #1101
Заменил рендер лишних контейнеров на React.Fragment.

Больше никаких div внутри span 🙈

@coveralls
Copy link

coveralls commented Mar 12, 2020

Coverage Status

Coverage remained the same at 85.248% when pulling b82d8a5 on refactor/simplify-amount-markup into 200befc on master.

@reme3d2y reme3d2y linked an issue Mar 12, 2020 that may be closed by this pull request
Copy link
Contributor

@Luchanso Luchanso left a comment

Choose a reason for hiding this comment

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

Только это BREAKING CHANGE - не забудьте выпустить пожалуйста мажорку после мержа

PS потому что некоторые стили могут быть завязаны на этот лишний span

@stepancar
Copy link
Contributor

@Luchanso в наших репо нашел 8 юсаджей.
Все проекты устаревшие и в любом случае требуют преехода на текущий мажор. Поэтоиму я предлагаю катить без мажора

@SiebenSieben SiebenSieben merged commit 9464a95 into master Apr 17, 2020
@SiebenSieben SiebenSieben deleted the refactor/simplify-amount-markup branch April 17, 2020 07:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Simplify amount implementation
5 participants