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

feat(markdown): code with links styles (#DS-3218) #473

Merged
merged 5 commits into from
Dec 11, 2024
Merged

Conversation

sk8knut
Copy link
Contributor

@sk8knut sk8knut commented Dec 5, 2024

No description provided.

@sk8knut sk8knut self-assigned this Dec 5, 2024
Copy link

github-actions bot commented Dec 5, 2024

Visit the preview URL for this PR (updated for commit a4c18f0):

https://koobiq-next--prs-473-dehz5jmk.web.app

(expires Sun, 15 Dec 2024 14:06:12 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@vitalyozza
Copy link
Contributor

vitalyozza commented Dec 5, 2024

@sk8knut посмотрел наработки, что заметил / хотел бы уточнить:

  1. Можно поправить шрифт, он у нас в этом компоненте mono и я в токенах это уже учел, подробнее можно почитать об этом вот тут: feat(typography): added new typo styles for our new inline code comp. design-tokens#66 (comment)
  2. Не нашел примерчика со сслылочкой, можешь пожалуйста указать, где это можно посмотреть?
  3. Интересный момент всплыл (картинка ниже): наложение верхнего inline-code на соседа снизу. Уже завтра это посмотрю в Figme, может быть просто уменьшу по высоте этот компонент.
    image

@@ -8,6 +8,7 @@ require('@koobiq/tokens-builder/transforms/attribute/palette.js')(StyleDictionar
require('@koobiq/tokens-builder/transforms/attribute/prefix.js')(StyleDictionary);
require('@koobiq/tokens-builder/transforms/attribute/font.js')(StyleDictionary);
require('@koobiq/tokens-builder/transforms/attribute/theme.js')(StyleDictionary);
require('@koobiq/tokens-builder/transforms/attribute/scss-value.js')(StyleDictionary);
Copy link
Contributor

Choose a reason for hiding this comment

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

Подтяни main, там этот фикс есть

@NikGurev
Copy link
Contributor

Инлайн код отображается без скруглений, так и должно быть?

image

@sk8knut sk8knut requested a review from NikGurev December 10, 2024 13:59
* main:
  fix(cdk): resolve event target within shadow-root (#DS-2632) (#475)
  feat(form-field): autofill backgrounds (#DS-2873) (#476)
  fix(docs): added build-tokens script missing transform (#474)
@vitalyozza
Copy link
Contributor

@sk8knut вижу radiuses по 8px, супер)

@sk8knut sk8knut merged commit 3605e00 into main Dec 11, 2024
3 checks passed
@sk8knut sk8knut deleted the feat/DS-3218 branch December 11, 2024 08:44
sk8knut added a commit that referenced this pull request Dec 16, 2024
* main:
  chore: remove vsix
  chore(docs): removed redundant code (#481)
  feat(timezone): select search fix (#DS-3213)  (#479)
  chore: bump version to 18.10.0 w/ changelog
  feat(docs): design-tokens page (#DS-2997) (#468)
  feat(markdown): code with links styles (#DS-3218) (#473)
  fix(cdk): resolve event target within shadow-root (#DS-2632) (#475)
  feat(form-field): autofill backgrounds (#DS-2873) (#476)
  fix(docs): added build-tokens script missing transform (#474)
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