microsoft/Terminal

Theme changes don't work correctly after opening the About box

Open

#12 093 ouverte le 5 janv. 2022

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-SettingsUIHelp WantedIssue-BugPriority-3Product-Terminal

Métriques du dépôt

Stars
 (35 764 stars)
Métriques de merge PR
 (Merge moyen 27j 19h) (24 PRs mergées en 30 j)

Description

Windows Terminal version

1.12.3472.0

Windows build number

10.0.19041.1415

Other Software

No response

Steps to reproduce

  1. Start Windows Terminal (don't just use an existing session).
  2. Open the Settings UI, and go to the Appearance section.
  3. Set the Theme to Light, and click Save.
  4. Without closing the settings, open the About box, and close it again.
  5. Now switch the Theme to Dark, and click Save.

Expected Behavior

The UI colors should change to the dark theme.

Actual Behavior

Most of the UI remains using the light theme, but some elements appear to be updated in such a way that they become invisible.

image

This breaks in a similar way when switching from dark to light, and it's also not only the About box that triggers it - I think any popup dialog will do (e.g. the "Do you want to close all tabs?" warning has the same effect).

If you open the About box again, it temporarily "fixes" the problem (i.e. it refreshes the colors correctly), but that only last until you try and change the theme again.

Guide contributeur

Direction de recherche
Examinez la logique de changement de thème dans l'interface utilisateur des paramètres, probablement dans `src/cascadia/TerminalPage.cpp`. Regardez la méthode ` UpdateTheme` et comment elle interagit avec les boîtes de dialogue contextuelles comme la boîte "À propos". Déterminez pourquoi l'ouverture d'une boîte de dialogue empêche les changements de thème ultérieurs de s'appliquer complètement. Portez attention à tout problème de mise en cache d'état ou de séquence d'événements.
Stack technique
cpp
Domaine
desktop
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70