RaspberryPiFoundation / editor-ui

Code Editor web component
https://editor-static.raspberrypi.org
Apache License 2.0
37 stars 8 forks source link

Tweaks to Web Component #964

Closed magdalenajadach closed 3 months ago

magdalenajadach commented 3 months ago

Closing: https://github.com/RaspberryPiFoundation/digital-editor-issues/issues/138

github-actions[bot] commented 3 months ago
github-actions[bot] commented 3 months ago
github-actions[bot] commented 3 months ago
github-actions[bot] commented 3 months ago
github-actions[bot] commented 3 months ago
github-actions[bot] commented 3 months ago
github-actions[bot] commented 3 months ago
loiswells97 commented 3 months ago

Looks like some of the styles are still missing, for example, the coloured underline beneath selected tab, the colours for the selected tab in the sidebar, download icon in dark mode, settings ticks in dark mode against the teal background should be black. Also, if we're having the default theme colour being teal, then I think we need to think about whether the buttons need to default to teal as well rather than blue 🤔

Screenshot 2024-03-15 at 11 29 32

github-actions[bot] commented 3 months ago
github-actions[bot] commented 3 months ago
magdalenajadach commented 3 months ago

@loiswells97 Those findings are all known. I should flag it before the review. So the issue is with theming between Projects Site embedded WC, standalone WC and Editor. When applying this teal color we encountered loads of issues with styling and themes. Since we don't yet have a very clear vision of dark mode for the standalone WC we decided to push it as it is, since the Standalone in a light (default) mode was not usable for online courses. So long story short it's an update and progress, but not yet for the dark mode due to theming shenanigans.

github-actions[bot] commented 3 months ago
loiswells97 commented 3 months ago

@loiswells97 Those findings are all known. I should flag it before the review. So the issue is with theming between Projects Site embedded WC, standalone WC and Editor. When applying this teal color we encountered loads of issues with styling and themes. Since we don't yet have a very clear vision of dark mode for the standalone WC we decided to push it as it is, since the Standalone in a light (default) mode was not usable for online courses. So long story short it's an update and progress, but not yet for the dark mode due to theming shenanigans.

Ah I see, in that case, I'm happy to approve - as you say, it's an improvement that should make it usable in light mode for the online courses at least 👍

github-actions[bot] commented 3 months ago
magdalenajadach commented 3 months ago

I managed to fix most of the issues with dark mode, removed peasky teal.

I achieved that overrriding some styles within InternalStyles.scss - it's not impacting Editor or Projectst site.

Still left some commenst to flag up what I have done.

magdalenajadach commented 3 months ago
Screenshot 2024-03-18 at 13 27 20 Screenshot 2024-03-18 at 13 27 38