Skip to content

Conversation

@stokito
Copy link
Contributor

@stokito stokito commented Nov 23, 2025

Before:
psi search dialog problem

As you see the group box title: "Search fields:" is above instructions text.

After:
psi search dialog fixed

I also tried to add a vertical splitter between search fields panel and search results but wasn't able

@Ri0n
Copy link
Member

Ri0n commented Nov 23, 2025

from the diff I can't understand anything, but if it looks good on low dpi too, then let's merge

@stokito
Copy link
Contributor Author

stokito commented Nov 24, 2025

You may see diff without spaces and identation changes. Click on a gear icon and select the "Hide whitespace".

The first commit adds names to components because it's difficult to understand what is where.
Also there was Vertical Box Layout inside of another that was useless:

       <layout class="QVBoxLayout">
        <item>
         <layout class="QVBoxLayout">
        <item>

To make it simpler I removed one VBox.

The second commit contains the fix where <property name="margin"> is replaced with four separate margins as the QT Designer did:

        <property name="leftMargin">
         <number>1</number>
        </property>
        <property name="topMargin">
         <number>10</number>
        </property>
        <property name="rightMargin">
         <number>1</number>
        </property>
        <property name="bottomMargin">
         <number>1</number>
        </property>

The only change here is increased topMargin to 10 px. It will be fine even on lower DPI.

I also tried to use the QSplitter but for some reason it doesn't work as expected. I tried a lot but wasn't able to make it work. Maybe you can do this, because the search fields panel may be too small.

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