Minor gui fixes#458
Conversation
Removed title from PSO edit widget since no other edit widget has one and title dublicates description bar label
| <bool>true</bool> | ||
| </property> | ||
| <property name="title"> | ||
| <string>Password Settings</string> |
There was a problem hiding this comment.
No need to remove that label
There was a problem hiding this comment.
ConsoleWidget has a description bar so when it's enabled "Password settings" shows up twice, which seems a bit excessive. And I believe this is the reason why any other result widget has a header
| <item> | ||
| <spacer name="verticalSpacer_7"> | ||
| <property name="orientation"> | ||
| <enum>Qt::Vertical</enum> |
There was a problem hiding this comment.
Have that parameter and the ones below been changed due to compiler warnings?
There was a problem hiding this comment.
QtCreator made this change on its own and I didn't even bother to check *.ui file change. I can remove this change or split it in another commit, but I believe QtCreator had its reasons to make these changes so we're better leave them be, and I have no idea what these reasons are thereof I can't make a good commit message for it. So I would prefer if we leave them as is
2 participants
No description provided.