Skip to content

Conversation

@Aga-C
Copy link
Member

@Aga-C Aga-C commented Mar 24, 2024

What is it?

  • Bugfix
  • Feature
  • Codebase improvement

Description of the changes in your PR

  • Added setting default tab in Settings.
  • Changed current tab consts to be similar as in other apps (next powers of 2). I needed to free the 0 for TAB_LAST_USED, so I decided to use the schema from other apps, maybe it will be useful in the future (if we'd like to have tab hiding).
  • There's a lot of relying on tab indexes, so I've added new consts for them.

Before/After Screenshots/Screen Record

qemu-system-x86_64_jB75BbCYnr.mp4

Fixes the following issue(s)

Acknowledgement

@naveensingh
Copy link
Member

Looks good now, thanks!

@naveensingh naveensingh merged commit c2a346a into FossifyOrg:master Jan 31, 2025
3 of 4 checks passed
@Aga-C Aga-C deleted the set-default-tab branch January 31, 2025 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature request] Add option to Settings to set default tab for opening

2 participants