Johani
25 يوليو 2022، 12:04م
21
لقد فعلنا ذلك، لكننا أغفلنا الحالة التي يحدد فيها المستخدم مخطط الألوان العادي فقط على أنه داكن. كنا نعتمد سابقًا على تفضيل نظام التشغيل للاختيار بين الوضع الفاتح/الداكن للشاشة الافتتاحية مع افتراض أن المستخدمين يحددون كليهما ويستخدمون مفتاح التبديل الخاص بنظام التشغيل.
لقد دفعت ببعض الإصلاحات
main ← hnb-ku:splash-background-flicker
opened 07:01AM - 23 Jul 22 UTC
Context: https://meta.discourse.org/t/introducing-discourse-splash-a-visual-prel… oader-displayed-while-site-assets-load/232003/17
We currently set the theme `secondary` color as the background for the splash, and this works and respects light/dark modes.
The issue is that we set it on the `#d-splash` div. That div doesn't have a specified height and only gets its height when the splash image loads.
This can cause a flicker effect where the `<HTML>` background shows for a fraction of a second while the splash image loads.
This PR sets the theme color on the `<HTML>` tag to alleviate this. This allows us to set the theme color a little bit sooner and should hopefully prevent the flicker effect from happening.
This PR also adds the `theme-color` `<meta>` tag for dark mode. Browsers that don't support multiple `theme-color` tags will ignore the second tag and fall back to the first one.
main ← hnb-ku:splash-flicker-take-2
opened 12:56PM - 23 Jul 22 UTC
Follow up to: https://github.com/discourse/discourse/pull/17619
Context: https:… //meta.discourse.org/t/introducing-discourse-splash-a-visual-preloader-displayed-while-site-assets-load/232003/17
We previously relied on the user's browser when deciding when to show the splash in light/dark mode. This worked well but can fail if the user manually selects a theme with a default "dark" scheme.
This PR will now factor that in. If the user selects a theme with a default dark scheme, use that. If a user selects a theme with a "light" default scheme and also picks a secondary "dark" scheme, use the `media` detection we had before.
This PR also removes the dark mode `theme-color` that was added in the previous PR. That will now go in a separate PR
إذا كان المخطط العادي داكنًا، فاستخدمه. إذا كان فاتحًا، فاستخدم المخطط الفاتح لوضع نظام التشغيل الفاتح والمخطط الداكن لوضع نظام التشغيل الداكن (إذا تم تحديده)
المشكلة، وكذلك هذه المشكلة
يجب أن يتم إصلاحه الآن. شكرًا على التقارير
11 إعجابًا
