Skip to content

Rename "night mode" to "dark theme" #20228

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
timabbott opened this issue Nov 10, 2021 · 14 comments
Closed

Rename "night mode" to "dark theme" #20228

timabbott opened this issue Nov 10, 2021 · 14 comments
Labels
area: settings (user) help wanted release goal We'd like to resolve this for the current major release

Comments

@timabbott
Copy link
Member

timabbott commented Nov 10, 2021

As discussed in #19728, we should rename "night mode" to "dark theme" across the UI. See a28c69c an incomplete first effort towards this.

Here's a proposal for what we change, each as its own commit:

  • Change "Automatic" to "Sync with computer" as the label for the "automatic" option in settings -- that's a lot more clear about what this it does for humans. We can keep the automatic internal value; that's clear enough for developers.
  • Rename the file night-mode.md in the Help Center to dark-theme.md, and include a redirect. 302aba0 is a sample similar commit. Use git grep night_mode.md to make sure we update all references in this commit. This commit should make no other changes.
  • Rename night-mode.css to dark-theme.css; use git grep night-mode.css to make sure we update all references.
  • Change "Night mode" => "Dark theme" everywhere it appears in text (use git grep -i "night mode" to find references). (skip locale, since those are generated translation files).
  • Change the handful of new "Dark mode" labels in the typeahead to "Dark theme", and similarly for the light one. And perhaps also change the descriptions to be something clearer like "Switch to the dark theme."

We'll probably want to do 5-10 more renaming commits afterwards, but I think that is a pretty long series for someone who may be new to building commit series like this, where one carefully only changes one class of thing in each commit to make one's work easy to verify and review.

@timabbott timabbott added help wanted area: settings (user) release goal We'd like to resolve this for the current major release labels Nov 10, 2021
@zulipbot
Copy link
Member

Hello @zulip/server-settings members, this issue was labeled with the "area: settings (user)" label, so you may want to check it out!

@andersk
Copy link
Member

andersk commented Nov 11, 2021

I thought the conclusion of https://chat.zulip.org/#narrow/stream/9-issues/topic/day.2Fnight.20slash.20commands was that we wanted to rename “night” to “dark”, not the other way around?

(Edited this in, since the previous text was inconsistent with itself.)

@andersk andersk changed the title Rename "dark mode" to "night theme" Rename "night mode" to "dark theme" Nov 11, 2021
@anjalivats2712
Copy link
Collaborator

@zulipbot claim

@zulipbot
Copy link
Member

Welcome to Zulip, @anjalivats2712! We just sent you an invite to collaborate on this repository at https://github.com/zulip/zulip/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!

Here's some tips to get you off to a good start:

As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!

@NerdyLucifer
Copy link
Collaborator

@timabbott may i work on this issue?

@timabbott
Copy link
Member Author

I think I'd like an experienced contributor to work on this since it's a release goal; @sahil839 can you take this issue?

@sahil839
Copy link
Collaborator

Sure.

sahil839 added a commit to sahil839/zulip that referenced this issue Nov 18, 2021
This commit renames "Automatic" option in color scheme setting
dropdown to "Sync with compute". We do not change any variables
used in code just the text in the dropdown visible to user.

Fixes part of zulip#20228.
sahil839 added a commit to sahil839/zulip that referenced this issue Nov 18, 2021
sahil839 added a commit to sahil839/zulip that referenced this issue Nov 18, 2021
sahil839 added a commit to sahil839/zulip that referenced this issue Nov 18, 2021
This commit renames "Night mode" to "Dark theme" in the
settings dropdown, help pages and api documentation.

Fixes part of zulip#20228.
sahil839 added a commit to sahil839/zulip that referenced this issue Nov 18, 2021
sahil839 added a commit to sahil839/zulip that referenced this issue Nov 18, 2021
sahil839 added a commit to sahil839/zulip that referenced this issue Nov 18, 2021
sahil839 added a commit to sahil839/zulip that referenced this issue Nov 18, 2021
timabbott pushed a commit to sahil839/zulip that referenced this issue Nov 19, 2021
This commit renames "Automatic" option in color scheme setting
dropdown to "Sync with compute". We do not change any variables
used in code just the text in the dropdown visible to user.

Fixes part of zulip#20228.
timabbott pushed a commit to sahil839/zulip that referenced this issue Nov 19, 2021
timabbott pushed a commit to sahil839/zulip that referenced this issue Nov 19, 2021
timabbott pushed a commit to sahil839/zulip that referenced this issue Nov 19, 2021
This commit renames "Night mode" to "Dark theme" in the
settings dropdown, help pages and api documentation.

Fixes part of zulip#20228.
timabbott pushed a commit to sahil839/zulip that referenced this issue Nov 19, 2021
timabbott pushed a commit to sahil839/zulip that referenced this issue Nov 19, 2021
@anjalivats2712 anjalivats2712 removed their assignment Nov 21, 2021
@timabbott
Copy link
Member Author

@sahil839 I think what's left may be mostly comments, but git grep -i dark[_ ]mode and similar things still find a good number of comments that need tweaking.

@sahil839
Copy link
Collaborator

Apart from comments, do we want to change the classes and ids of elements? And also the js file is still named as night_mode.

@timabbott
Copy link
Member Author

I'd at least rename the file to dark_theme.ts, and if it's easy, change all the classes/IDs as well. I don't think we're likely to undo the changes, and it's just technical debt having both names floating around.

danielpyon pushed a commit to danielpyon/zulip that referenced this issue Dec 1, 2021
This commit renames "Automatic" option in color scheme setting
dropdown to "Sync with compute". We do not change any variables
used in code just the text in the dropdown visible to user.

Fixes part of zulip#20228.
danielpyon pushed a commit to danielpyon/zulip that referenced this issue Dec 1, 2021
danielpyon pushed a commit to danielpyon/zulip that referenced this issue Dec 1, 2021
danielpyon pushed a commit to danielpyon/zulip that referenced this issue Dec 1, 2021
This commit renames "Night mode" to "Dark theme" in the
settings dropdown, help pages and api documentation.

Fixes part of zulip#20228.
danielpyon pushed a commit to danielpyon/zulip that referenced this issue Dec 1, 2021
danielpyon pushed a commit to danielpyon/zulip that referenced this issue Dec 1, 2021
@timabbott
Copy link
Member Author

@sahil839 is there anything else left for this issue?

@sahil839
Copy link
Collaborator

sahil839 commented Dec 3, 2021

Opened #20441 for a minor renaming of tests.

We use variables like realm_night_logo_source, realm_night_logo_url. Basically we use night in some variables in the realm-logo related code. Other than this, we have renamed everything.

@timabbott
Copy link
Member Author

Yeah, I think that's maybe worth discussing in #api design whether it's worth renaming; I can see arguments either way. Want to start a thread?

@timabbott
Copy link
Member Author

I think we can close this with #20441; and just decide if we want the little API churn there -- we might since I think those fields are likely only used by the web app at this point, and thus it's cheap to clean up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: settings (user) help wanted release goal We'd like to resolve this for the current major release
Projects
None yet
Development

No branches or pull requests

6 participants