Skip to content

Various fixes#1636

Merged
ge0rdi merged 5 commits into
masterfrom
fixes
Aug 16, 2023
Merged

Various fixes#1636
ge0rdi merged 5 commits into
masterfrom
fixes

Conversation

@ge0rdi

@ge0rdi ge0rdi commented Jul 31, 2023

Copy link
Copy Markdown
Member

ge0rdi added 2 commits July 31, 2023 13:50
It will be now separate from Explorer's icons in taskbar.
Open-Shell icon will be displayed for better clarity.

#31 (comment)
Menu offsets were previously applied to s_MainMenuLimits.
Though when deciding where to put start menu (CalculateCorner) we are
now taking into account also position of start button (as since Win11 it
may not be in corner of the screen).

This broke negative menu offsets.

We will fix it by applying menu offsets in CalculateCorner as a last
step once we have determined start menu position.
This should make even more sense than previous approach.
@ge0rdi ge0rdi changed the title Varipous fixes Various fixes Jul 31, 2023
@AppVeyorBot

Copy link
Copy Markdown

@ge0rdi

ge0rdi commented Jul 31, 2023

Copy link
Copy Markdown
Member Author

@AEAEAEAE4343
Hello.
You have introduced settings for horizontal/vertical start menu offset in #620 PR.
I had to adjust it a bit. Basically applying those offsets a bit later when we decided on position of start menu.

Could you eventually check if such change still works for you?

@AppVeyorBot

Copy link
Copy Markdown

@leetftw

leetftw commented Aug 1, 2023

Copy link
Copy Markdown

The PR looks good as far as I can tell. As long as the option behaves as expected, I’m fine with the changes.

@ge0rdi

ge0rdi commented Aug 1, 2023

Copy link
Copy Markdown
Member Author

@AEAEAEAE4343

As long as the option behaves as expected, I’m fine with the changes.

Well, that's what I wanted you to confirm :)
From my POV it seems to be working.

Changes in be8568 commit caused that under certain circumstances search
results may overlap "Search the Internet" item.
@AppVeyorBot

Copy link
Copy Markdown

Commit be8568c that introduced option to disable "See more results" also
changed behavior of "Searching..." item that indicates search in progress.

We will now make sure that the search progress indicator is displayed
always. No matter of options.
@AppVeyorBot

Copy link
Copy Markdown

@ge0rdi
ge0rdi merged commit c517e21 into master Aug 16, 2023
@ge0rdi
ge0rdi deleted the fixes branch August 16, 2023 13:39
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.

3 participants