Skip to content

🐞 Changes to the color scheme are not applied in real-time #990

Closed

Description

@Qubik65536

Description

When we are changing the color scheme for a particular type of text, it does not change until we switch to another theme and switch it back.

To Reproduce

  1. Open CodeEdit
  2. Open a file with syntax highlighting
  3. Go to Settings, Themes
  4. Change the color scheme of a type of text (e.g. Keywords)
  5. Go back to the editor, the color doesn’t change
  6. Change to another theme
  7. Change back the theme
  8. The changed color is now applied

2023-01-17_10-12-22.mp4

Expected behavior

When we change the color scheme, it should instantly apply.

Version information

CodeEdit: 78bf33e
macOS: 13.1 (22C65)
Xcode: 14.1 (14B47b)

Additional context

N/A

Activity

  1. austincondiff commented on Jan 18, 2023

    @austincondiff
    Collaborator

    Nice find! Color values should really be updating in real-time as the user changes them so they can get instant feedback.

  2. changed the title [-]🐞 The color scheme changes don't apply instantly[/-] [+]🐞 Changes to the color scheme or font are not applied in real-time[/+] on Jan 18, 2023
  3. Qubik65536 commented on Jan 18, 2023

    @Qubik65536
    ContributorAuthor

    I was looking at these problems, and from what I can tell, there's just no function taking charge of the color scheme changes, so, the changes on the theme won't be applied to selectedTheme variable, so the only way to apply the changes has to change the theme and change it back (as the color value is applied, but the selectedTheme should be reloaded).

  4. ben-p-commits commented on Jan 18, 2023

    @ben-p-commits
    Contributor

    @Cubik65536- font updates are being handled in a PR that just merged into CodeEditTextView:
    CodeEditApp/CodeEditTextView#122

    You're right about the live color scheme changes needing some additional love to address..

  5. ben-p-commits commented on Jan 18, 2023

    @ben-p-commits
    Contributor

    @Cubik65536 - can you modify the scope of this issue to remove the concern about fonts, to keep things tidy / separate?

  6. Qubik65536 commented on Jan 18, 2023

    @Qubik65536
    ContributorAuthor

    @Cubik65536 - can you modify the scope of this issue to remove the concern about fonts, to keep things tidy / separate?

    I will do it later

  7. changed the title [-]🐞 Changes to the color scheme or font are not applied in real-time[/-] [+]🐞 Changes to the color scheme are not applied in real-time[/+] on Jan 18, 2023
  8. Qubik65536 commented on Jan 18, 2023

    @Qubik65536
    ContributorAuthor

    @Cubik65536 - can you modify the scope of this issue to remove the concern about fonts, to keep things tidy / separate?

    I did the modification and removed the concern about fonts.

  9. ben-p-commits commented on Jan 24, 2023

    @ben-p-commits
    Contributor

    @austincondiff - this rhymes with some of the stuff I did around line wrapping / misc text editor prefs. would be happy to nab this one.

  10. austincondiff commented on Feb 16, 2023

    @austincondiff
    Collaborator

    @ben-p-commits what is the status of this one?

  11. moved this from 🆕 New to 🏃‍♂️ In Progress in CodeEdit Projecton Feb 16, 2023
  12. austincondiff commented on Apr 19, 2025

    @austincondiff
    Collaborator

    This works now, closing.

    Screen.Recording.2025-04-18.at.11.33.25.PM.mov
  13. moved this from 🏃‍♂️ In Progress to 🏁 Complete in CodeEdit Projecton Apr 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingtheme

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions