Skip to content
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

New settings menu 'Appearance' #7986

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

cintek
Copy link
Contributor

@cintek cintek commented Jan 20, 2023

Type of change

  • Feature
  • Bugfix
  • Technical
  • [] Other :

Content

This PR adds a new entry in the settings: Appearance
As you can see on the screenshots settings for theme, font and message bubbles are moved to the new menu and the settings for the language is moved to "General".

Solves #7889

Motivation and context

In #7889 I explained why I wanted to change the settings.

Screenshots / GIFs

Root Settings

Appearance Settings (New)

General Settings (Added language)

Tests

Changed the settings for language, theme, font, bubbles to see if everything still works.

Tested devices

  • Physical
  • Emulator
  • OS version(s):

Checklist

Signed-off-by: Christoph Klassen [email protected]

@cintek cintek force-pushed the appearance_settings branch from 1007d18 to 8d7481a Compare January 20, 2023 16:15
@cintek cintek force-pushed the appearance_settings branch from 8d7481a to fdc0210 Compare January 20, 2023 16:16
@bmarty bmarty added the Z-Community-PR Issue is solved by a community member's PR label Jan 23, 2023
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants