Feat/markdown: split inner component desktop / mobile

[denisx]

Markdown : cплит внутреннего компонента десктоп / мобайл

[changeset-bot]

🦋 Changeset detected

Latest commit: bcb2c9a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@alfalab/core-components	Patch
@alfalab/core-components-markdown	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[coveralls]

Pull Request Test Coverage Report for Build 17916997489

Details

• 14 of 14 (100.0%) changed or added relevant lines in 5 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.009%) to 79.746%

---

Totals
Change from base Build 17868060226:	0.009%
Covered Lines:	9671
Relevant Lines:	11363

---

💛 - Coveralls

[github-actions]

Demo build

https://core-ds.github.io/core-components/1891

[fulcanellee]

В кодовой базе библиотеки любой base компонент подразумевается в единственном экземпляре. Текущий PR просто размножил этот базовый компонент, ради того, чтобы разделить типографику. Текущие решение считаю избыточным.

[denisx]

В кодовой базе библиотеки любой base компонент подразумевается в единственном экземпляре. Текущий PR просто размножил этот базовый компонент, ради того, чтобы разделить типографику. Текущие решение считаю избыточным.

Если типографика не base, то как зависимые от неё компоненты могут быть монолитны?

Да и компоненты остался на месте. Просто у него добавился пропс. Пропсы в любом случае нужно передавать. Если ему нужен responsive импорт, давай доделаю, чтобы был base resposive (я просто думал, что импорты компонентов у нас только из корня пакетов, и внутряшки не требуют респонсива)

[fulcanellee]

Нужную типографику ты мог передать тут, не создавая BaseMarkdownDesktop и BaseMarkdownMobile

[dHIM24]

Поддерживаю позицию @fulcanellee , как будто практической пользы доработка не несет

[denisx]

да, просмотрел этот момент. упростил пр

[fulcanellee]

Отлично. Еще измени базовую ветку PR'a. next предназначена для мажорки. Также отметил пару моментов ниже.

[fulcanellee]

Не забудь next на master поменять. Нужно будет зачистить ветку до состояния мастера, а накатить изменения git патчем

[denisx]

@fulcanellee запутался. ставлю деприкейт на platform, и ветку на мастер (вар.1)
или оставляю как мажорку, и в next? (вар.2)

[fulcanellee]

@fulcanellee запутался. ставлю деприкейт на platform, и ветку на мастер (вар.1) или оставляю как мажорку, и в next? (вар.2)

я удалил тот коммент. сделал коммит. Делаем omit Title и выпускаем как патч от мастера