Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Welcome screen redesign #292

Merged
merged 2 commits into from
Feb 11, 2017
Merged

Welcome screen redesign #292

merged 2 commits into from
Feb 11, 2017

Conversation

TheZ3ro
Copy link
Contributor

@TheZ3ro TheZ3ro commented Feb 11, 2017

Description

Welcome Screen redesign

Motivation and Context

A nicer welcome screen for new users

How Has This Been Tested?

Manually on Linux (should work no problem on other platforms)

Screenshots (if appropriate):

istantanea_2017-02-11_16-37-23

Types of changes

  • ✅ New feature (non-breaking change which adds functionality)

Checklist:

  • ✅ I have read the CONTRIBUTING document. [REQUIRED]
  • ✅ My code follows the code style of this project. [REQUIRED]
  • ✅ All new and existing tests passed. [REQUIRED]

@@ -18,13 +18,52 @@
#include "WelcomeWidget.h"
#include "ui_WelcomeWidget.h"

#include <QDebug>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TheZ3ro I guess that was used for debugging while coding the feature?

Copy link
Contributor Author

@TheZ3ro TheZ3ro Feb 11, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wooops, I will force push

@phoerious
Copy link
Member

One more thing: You should either spell Database with a lowercase D or make everything title case.

@TheZ3ro TheZ3ro force-pushed the feature/redesign-welcome-screen branch from 749d250 to afd1fac Compare February 11, 2017 18:31
@louib
Copy link
Member

louib commented Feb 11, 2017

This is the UI on MacOs.

screen shot 2017-02-11 at 1 31 13 pm

The Import from KeePass1 button is not the same size as the others.

Also, maybe we could hide the Recent databases if the list of recent databases is empty.

Good job @TheZ3ro !!

@TheZ3ro TheZ3ro force-pushed the feature/redesign-welcome-screen branch from afd1fac to ee981c4 Compare February 11, 2017 18:35
@phoerious
Copy link
Member

I have the feeling that widget sizes on OS X are quite messed up in general. The keyfile dropdown in the unlock dialog is also not the same size as the password field. And the password fields in the new database widget are tiny. I think we need to polish all that a little for 2.2.0.

@TheZ3ro
Copy link
Contributor Author

TheZ3ro commented Feb 11, 2017

Oh, I know why the KeePass1 button is smaller. It's inside a horizontal layout with the (hide) CSV button.

@TheZ3ro TheZ3ro mentioned this pull request Feb 11, 2017
<brush brushstyle="SolidPattern">
<color alpha="255">
<red>0</red>
<green>255</green>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would remove that color. The button should have the system's default button color.
The green is really lurid and also very surprising. When the button has focus, it has a nice light blue on my system, but when it loses focus, it's suddenly shocking green.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems that this wasn't reverted in the previus force-push

<item>
<widget class="QPushButton" name="buttonImportKeePass1">
<property name="text">
<string>Import from KeePass1</string>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The file filter in the open dialog has a space between "KeePass" and "1". I suggest adding one here, too.

@TheZ3ro TheZ3ro merged commit 7f51160 into develop Feb 11, 2017
@TheZ3ro TheZ3ro deleted the feature/redesign-welcome-screen branch February 11, 2017 23:07
droidmonkey added a commit that referenced this pull request Jun 25, 2017
- Added YubiKey 2FA integration for unlocking databases [#127]
- Added TOTP support [#519]
- Added CSV import tool [#146, #490]
- Added KeePassXC CLI tool [#254]
- Added diceware password generator [#373]
- Added support for entry references [#370, #378]
- Added support for Twofish encryption [#167]
- Enabled DEP and ASLR for in-memory protection [#371]
- Enabled single instance mode [#510]
- Enabled portable mode [#645]
- Enabled database lock on screensaver and session lock [#545]
- Redesigned welcome screen with common features and recent databases [#292]
- Multiple updates to search behavior [#168, #213, #374, #471, #603, #654]
- Added auto-type fields {CLEARFIELD}, {SPACE}, {{}, {}} [#267, #427, #480]
- Fixed auto-type errors on Linux [#550]
- Prompt user prior to executing a cmd:// URL [#235]
- Entry attributes can be protected (hidden) [#220]
- Added extended ascii to password generator [#538]
- Added new database icon to toolbar [#289]
- Added context menu entry to empty recycle bin in databases [#520]
- Added "apply" button to entry and group edit windows [#624]
- Added macOS tray icon and enabled minimize on close [#583]
- Fixed issues with unclean shutdowns [#170, #580]
- Changed keyboard shortcut to create new database to CTRL+SHIFT+N [#515]
- Compare window title to entry URLs [#556]
- Implemented inline error messages [#162]
- Ignore group expansion and other minor changes when making database "dirty" [#464]
- Updated license and copyright information on souce files [#632]
- Added contributors list to about dialog [#629]
@phoerious phoerious added pr: new feature Pull request that adds a new feature and removed new feature labels Nov 22, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
pr: new feature Pull request that adds a new feature user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants