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

Add navigation bar #308

Closed
wants to merge 9 commits into from
Closed

Add navigation bar #308

wants to merge 9 commits into from

Conversation

Reckless-Satoshi
Copy link
Collaborator

What does this PR do?

Approaching the end of the re-do of the basic frontend. This PR introduces the new Navigation Bar and the Settings Page. Playing with sliding pages, looks good but sometimes behaves unexpectedly if device is slow. Final decision will be made based in performance.

Still has the old UserGenPage in place.

Checklist before merging

  • Make sure you have installed and initialized pre-commit. pip install pre-commit and pre-commit install

@Reckless-Satoshi Reckless-Satoshi marked this pull request as ready for review October 30, 2022 11:01
@KoalaSat
Copy link
Member

KoalaSat commented Oct 30, 2022

what a nice clean up 😄 couldn't check everything in details but certainly it improves the general structure congrats!

@Reckless-Satoshi
Copy link
Collaborator Author

Will be merging as is. I have not tested fully and some things might be broken: but things are broken in main anyway and there is a lot of fixes here! 😅

Reckless-Satoshi added a commit that referenced this pull request Oct 30, 2022
commit a5b63ae
Author: Reckless_Satoshi <reckless.satoshi@protonmail.com>
Date:   Sun Oct 30 10:46:05 2022 -0700

    Small fixes

commit d64adfc
Author: Reckless_Satoshi <reckless.satoshi@protonmail.com>
Date:   Sun Oct 30 06:02:06 2022 -0700

    wip work on federation settings

commit ca35d6b
Author: Reckless_Satoshi <reckless.satoshi@protonmail.com>
Date:   Sun Oct 30 04:05:33 2022 -0700

    Refactor confirmation Dialogs

commit c660a5b
Author: Reckless_Satoshi <reckless.satoshi@protonmail.com>
Date:   Sat Oct 29 13:36:59 2022 -0700

    refactor login (clean separation robot/info. Style navbar.

commit b9dc7f7
Author: Reckless_Satoshi <reckless.satoshi@protonmail.com>
Date:   Fri Oct 28 09:54:38 2022 -0700

    Add size slider and settings widget

commit 20b2b3d
Author: Reckless_Satoshi <reckless.satoshi@protonmail.com>
Date:   Fri Oct 28 05:41:48 2022 -0700

    Add show more and Dialogs

commit da8b700
Author: Reckless_Satoshi <reckless.satoshi@protonmail.com>
Date:   Thu Oct 27 16:26:07 2022 -0700

    Add sliding pages

commit 6dd90aa
Author: Reckless_Satoshi <reckless.satoshi@protonmail.com>
Date:   Thu Oct 27 06:34:58 2022 -0700

    Add settings forms

commit d3d0f3e
Author: Reckless_Satoshi <reckless.satoshi@protonmail.com>
Date:   Wed Oct 26 04:16:06 2022 -0700

    Refactor utils
@Reckless-Satoshi
Copy link
Collaborator Author

Merged via CLI

@Reckless-Satoshi Reckless-Satoshi deleted the add-navigation-bar branch October 31, 2022 22:14
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.

2 participants