Skip to content

Adjust Max Width Window Size #1355

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 6, 2022
Merged

Adjust Max Width Window Size #1355

merged 2 commits into from
Sep 6, 2022

Conversation

@onesounds
Copy link
Contributor

@onesounds onesounds commented Sep 1, 2022 •
edited
Loading

  • Change Max Width to 1920 from 900 (Window Size Slider in setting window)
  • Adjust Result Item Layout

Check List

  • Even if you increase the window size, the item layout must be displayed normally.
  • The shortcut (alt+1 things) display should be displayed normally when turned on and off.

ETC

  • 1920 is a temporary. I hope the width will be changed based on the Monitor's Width.

@onesounds
- Adjust ResultListItem Layout for Large Window Size
a2d9cb6 
- Change Window Max Width to 1920 from 900
@onesounds onesounds self-assigned this Sep 1, 2022
@onesounds onesounds added the kind/ui related to UI, icons, themes, etc label Sep 1, 2022
@onesounds onesounds changed the title Adjust Max Window Width Adjust Max Width Sep 1, 2022
@onesounds onesounds changed the title Adjust Max Width Adjust Max Width Window Size Sep 1, 2022
taooceros
taooceros reviewed Sep 1, 2022
View reviewed changes
Flow.Launcher/ResultListBox.xaml Outdated
HorizontalAlignment="Stretch"
VerticalAlignment="Stretch"
Cursor="Hand"
ShowGridLines="True"
Copy link
Member

@taooceros taooceros Sep 1, 2022

Choose a reason for hiding this comment

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

What's this line for?

Copy link
Contributor Author

@onesounds onesounds Sep 1, 2022

Choose a reason for hiding this comment

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

oh, my mistake. it is guide line for design. I'll remove.

@jjw24 jjw24 added review in progress Indicates that a review is in progress for this PR bug Something isn't working and removed review in progress Indicates that a review is in progress for this PR labels Sep 6, 2022
@jjw24 jjw24 added this to the 1.10.0 milestone Sep 6, 2022
@jjw24 jjw24 merged commit 05044ae into Flow-Launcher:dev Sep 6, 2022
@jjw24 jjw24 modified the milestones: 1.10.0, 1.9.5 Sep 16, 2022
jjw24 added a commit that referenced this pull request Sep 16, 2022
@jjw24
Merge pull request #1355 from onesounds/LimitWidth
c0bc936 
Adjust Max Width Window Size
@jjw24 jjw24 mentioned this pull request Sep 18, 2022
Merged
jjw24 added a commit that referenced this pull request Sep 27, 2022
@jjw24
Release 1.9.5 (#1386)
43ac9b4 
* Merge pull request #1061 from Flow-Launcher/remove_winget_ci

* Merge pull request #991 from Flow-Launcher/context_menu_plugin_site

* Merge pull request #1080 from gissehel/caret-position-fix

* Caret position fix : Include PR #1074

* Merge pull request #1283 from nachmore/dev

* Merge pull request #1296 from nachmore/bug_1284

* Merge pull request #1294 from Flow-Launcher/pluginInfoMultipleActionKeyword

* Merge pull request #1299 from nachmore/bug_1269

* Plugin Exception Draft (#1147)

* Merge pull request #1355 from onesounds/LimitWidth

* Merge pull request #1088 from Flow-Launcher/add_spanish_latin_america

* Merge pull request #1387 from Flow-Launcher/fix_exception_duplicate_url_opening

* Merge pull request #1390 from Flow-Launcher/issue_1371

* Merge pull request #1391 from Flow-Launcher/issue_1366
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@taooceros taooceros taooceros approved these changes

Assignees

@onesounds onesounds

Labels

bug Something isn't working kind/ui related to UI, icons, themes, etc

Projects

Work In Progress
Archived in project

Milestone

1.9.5

Development

Successfully merging this pull request may close these issues.

3 participants

@onesounds @taooceros @jjw24