Skip to content
Snippets Groups Projects

Themes 3: Intermission: Appearance Tab and fixes

Merged HJ requested to merge appearance-tab into develop
1 unresolved thread

Done:

  • reorganized settings a bit, created "appearance" tab to put all the visual (and not really functional) stuff in there.
  • fixed firefox+redmond theme bug
  • added UI scaling options
  • added UI roundness options
  • added emoji scale options
  • fix existing theme editor's preview pane
  • many fixes to emoji picker, now scalable with rest of emoji, further customizations of it - later
  • themes selector with preview
  • add option to make underlay solid or transparent
  • add option to override font
Edited by HJ

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • HJ resolved all threads

    resolved all threads

  • HJ added 1 commit

    added 1 commit

    • 169f71c0 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

    • Here's how it looks right after starting, everything is kinda huge.

      Screenshot_2024-07-23_at_13-12-48_シグセグV

      Preview doesn't look right Screenshot_2024-07-23_at_13-03-19_lain.com

      Clicking the up/down button will immediately bring up this modal. Accepting it will fix the visual weirdness. Screenshot_2024-07-23_at_13-05-04__3__lain.com

      The message here probably needs some top padding, but in general I find it very confusing that this tab even exists. I think it should go away completely or at least be hidden behind 'advanced'. Screenshot_2024-07-23_at_13-07-19_lain.com

      I do not understand what the right way to edit a theme is right now. Is it in the "Appearance" tab?

      The input fields seem to be too close to the labels, and the tabs seem to overflow into the main windows.

      Screenshot_2024-07-23_at_13-17-58_シグセグV

    • Author Maintainer

      I do not understand what the right way to edit a theme is right now. Is it in the "Appearance" tab?

      I find it very confusing that this tab even exists. I think it should go away completely or at least be hidden behind 'advanced'.

      Right way to edit theme is still in Theme tab. Maybe I should change the message to "will be deprecated" or something like this. There's currently no way to hide/show tabs depending on "advanced mode" state and I don't think it's worth implementing it yet. I will be working on actual Theme v3 format and editor that replaces current one later this year (i need a vacation)

      Edited by HJ
    • Author Maintainer

      The input fields seem to be too close to the labels, and the tabs seem to overflow into the main windows.

      "tabs overflowing into main window" is intentional, to give the appearance of them being... tabs, and not just buttons.

      Spacing between label and input - yeah I think i can add a space there, i'm just not entirely sure about accessibility.

    • Even knowing that the tabs are supposed to look like 'tabs', they don't look like it. Something about them is off. That's not an issue for this MR though.

      Also, I think Appearance and Theme should be right next to each other, as they are semantically related.

    • Author Maintainer

      yeah I can move them. Tabs have been like this for years, I don't know how to implement them better currently.

    • Author Maintainer

      Spacing between label and input - yeah I think i can add a space there, i'm just not entirely sure about accessibility.

      space has been there for other types of "setting" so it was just missing from UnitSetting and NumberSetting - fixed.

      Also, I think Appearance and Theme should be right next to each other, as they are semantically related.

      Re-shufffled tabs a bit:
      image image

      Better?

    • Please register or sign in to reply
  • HJ added 1 commit

    added 1 commit

    • 29289591 - fix huge previews on firefox

    Compare with previous version

    • Author Maintainer
      Resolved by HJ

      from @tusooa @ IRC:

      the buttons in the settings modal no longer have spacing in your appearance-tab branch

  • HJ added 3 commits

    added 3 commits

    • f4544f07 - elaboration and spacing
    • 5d78d12c - elaborates, refuses to leave
    • 355bcc49 - increase gap in settings modal titlebar buttons

    Compare with previous version

  • HJ added 1 commit

    added 1 commit

    • ece38e68 - spacing for number setting as well

    Compare with previous version

  • Author Maintainer

    @lambadalambda @tusooa i fixed the issues you mentioned (i think)

  • HJ added 1 commit

    added 1 commit

    • 5a929db0 - fixed lack of gap in panel headers

    Compare with previous version

  • HJ added 1 commit

    added 1 commit

    Compare with previous version

  • Author Maintainer

    just noticed there is some problem with a hide_muted_federation_restrictions string but that's in develop, will fix in a separate MR

  • HJ mentioned in commit 0c9893c8

    mentioned in commit 0c9893c8

  • merged

  • Please register or sign in to reply
    Loading