Skip to content

BREAKING CHANGE: remove scheme#2471

Merged
SevereCloud merged 2 commits intov5.0.0from
SevereCloud/useAutoAppearance
May 31, 2022
Merged

BREAKING CHANGE: remove scheme#2471
SevereCloud merged 2 commits intov5.0.0from
SevereCloud/useAutoAppearance

Conversation

@SevereCloud
Copy link
Copy Markdown
Contributor

@SevereCloud SevereCloud commented May 7, 2022

  • Выпилено все что связано с scheme(оставлен минимум для определения свойства appearance)
  • Оставлен минимум от Appearance
  • Добавлено автоопределение appearance в ConfigProvider по умолчанию
    • Проверяем appearance через vk-bridge, если доступен
    • Проверяем системную тему через медиа запрос
  • В full режиме класс vkui-tokens теперь вешается на html

@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci bot commented May 7, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 1f31dd5:

Sandbox Source
VKUI - default example Configuration

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 7, 2022

size-limit report 📦

Path Size
JS 74.9 KB (-0.14% 🔽)
JS (brotli) 63.06 KB (-0.26% 🔽)
JS, unstable 29.34 KB (-0.31% 🔽)
CSS 39.01 KB (+0.18% 🔺)
CSS (brotli) 31.35 KB (-0.07% 🔽)
CSS, unstable 1.05 KB (0%)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 7, 2022

👀 Styleguide deployed

See the styleguide for this PR at https://vkcom.github.io/VKUI/pull/2471/

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 7, 2022

Code coverage

lines3615 / 463178.06%
statements3683 / 471778.07%
functions829 / 103080.48%
branches2995 / 425670.37%
branchesTrue0 / 0100.00%

Generated by 🚫 dangerJS against 1f31dd5

@eolme
Copy link
Copy Markdown
Contributor

eolme commented May 8, 2022

Во избежание мигания лучше отрефачить: сначала показываем по prefers, потом по bridge, и не устанавливать в light по-умолчанию, а только если мы не можем никакими другими способами определить

@worldwidebaby
Copy link
Copy Markdown

Чтобы избежать мигание, лучше запрашивать первоначальное состояние через VKWebAppGetConfig, и вне тела функции. Не у всех пользователей стоит автоматическое определение темы.

@SevereCloud SevereCloud force-pushed the SevereCloud/useAutoAppearance branch 3 times, most recently from 29dd401 to be32982 Compare May 17, 2022 19:50
@SevereCloud SevereCloud changed the title Add auto detection appearance BREAKING CHANGE: remove scheme May 17, 2022
@SevereCloud SevereCloud force-pushed the SevereCloud/useAutoAppearance branch 3 times, most recently from e8c0240 to 33571f3 Compare May 18, 2022 07:28
@SevereCloud SevereCloud marked this pull request as ready for review May 18, 2022 08:01
@SevereCloud SevereCloud requested review from a team as code owners May 18, 2022 08:01
@inomdzhon inomdzhon linked an issue May 18, 2022 that may be closed by this pull request
@SevereCloud SevereCloud force-pushed the SevereCloud/useAutoAppearance branch 4 times, most recently from a944c47 to 156fdca Compare May 20, 2022 07:13
@SevereCloud SevereCloud force-pushed the SevereCloud/useAutoAppearance branch 2 times, most recently from a875bdc to fcbac6f Compare May 24, 2022 11:49
@SevereCloud SevereCloud force-pushed the SevereCloud/useAutoAppearance branch from fcbac6f to 73de722 Compare May 24, 2022 13:52
@SevereCloud SevereCloud marked this pull request as draft May 26, 2022 11:54
@SevereCloud SevereCloud force-pushed the SevereCloud/useAutoAppearance branch from 73de722 to e1dc693 Compare May 31, 2022 13:22
@SevereCloud SevereCloud marked this pull request as ready for review May 31, 2022 14:52
@SevereCloud SevereCloud force-pushed the SevereCloud/useAutoAppearance branch from e1dc693 to 1f31dd5 Compare May 31, 2022 15:44
Copy link
Copy Markdown
Contributor

@inomdzhon inomdzhon left a comment

Choose a reason for hiding this comment

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

🔥 👍 🚀

@SevereCloud SevereCloud merged commit 43febd8 into v5.0.0 May 31, 2022
@SevereCloud SevereCloud deleted the SevereCloud/useAutoAppearance branch May 31, 2022 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Breaking changes] Удаляем scheme

5 participants