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

Flex gui #1

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

Flex gui #1

wants to merge 22 commits into from

Conversation

dalnefre
Copy link
Collaborator

Proposed overhaul of GUI and clean up internal logic.

dalnefre and others added 22 commits August 3, 2022 12:40
It's just too much fun watching the site password change as you type to cover it up by default.
I originally had a single set of instructions for both the web page and the extension.  This update provides instructions specific to the web page.
The Remember button should be enabled when the domain name, site name, and user name fields are filled in.  It was also requiring the master password field have a value, which is an error.
There may be times when you want to want to forget the settings for a given domain name.  When you click the forget button, the domain name is removed from database.domains and the site name is removed from database.sites if there are no more references to it.
Seems more direct, doesn't it?
Maintain a cached string of the settings loaded from the database
in order to determine if/when the settings data-model has changed.
Use this to implement settingsModified(), which influences the state
of the Remember and Forget buttons in the GUI.
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