Skip to content

Conversation

onesounds
Copy link
Contributor

@onesounds onesounds commented Oct 9, 2021

  • Redesign Plugin page layout
  • Added Plugin Store page
  • Redesigned Plugin Priority Window pop out
  • Updated setting window's label wording
  • Removed auto hiding of scrollbar in setting window

@onesounds
Copy link
Contributor Author

onesounds commented Oct 9, 2021

Todo

  • organize and placing version / init and query time / Action Keyword
  • Adjust Plugin's margin
  • If there's no setting page, hide separator
  • Add Page Title
  • Change Expander toggle button design (Remove Circle)
  • Change to button. Priority / plugin folder / Action Keyword
  • Responsive sidebar width

Check if it's possible

  • Change priority control to Number box
  • Expand/collapse animation
  • Smooth Scroll
  • Control styling in plugin setting

It would be nice if each plug-in had an example query description. (it's not plugin description)

onesounds and others added 20 commits October 9, 2021 20:47
- Priority Button color change by Priority number (0 gray, 0<= Bold Black)
- Priority button update when property changed.
- Fix layout for init/query time/version/plugin store
- change action keyword to button
- add listbox bottom margin
- add margin when select list item for easy to distinguish.
- Adjust listbox Margin for fit other setting page list.
- Adjust Calc/webseearch plugin setting Margin
- Adjust Calc/webseearch plugin setting Margin
- Adjust Calc/webseearch plugin setting Margin
Use sender to find actual pluginviewmodel instead of use selected
@onesounds
Copy link
Contributor Author

onesounds commented Oct 10, 2021

image

image

  • I think install button need 4 state. 'install / installing... / update / open (quick move to plugin setting page)'
  • Uninstall is managed on the plug-in page.
  • Check that the version has changed during refresh, and if there is anything to update, order it to the top (and add little green icon)

Comment on lines 34 to 38
<system:String x:Key="autoHideScrollBar">Auto Hide Scroll Bar</system:String>
<system:String x:Key="autoHideScrollBarToolTip">Automatically hides the Settings window scroll bar and show when hover the mouse over it</system:String>
<system:String x:Key="autoHideScrollBar">Auto Hide Scroll Bar in Setting</system:String>
Copy link
Member

Choose a reason for hiding this comment

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

we dont need this right? since auto hide scroll bar is removed

Copy link
Member

Choose a reason for hiding this comment

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

i am still seeing alot of places where the auto hide scroll bar code exists, are we removing this option completely? if so i can clean it up.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we dont need this right? since auto hide scroll bar is removed

This function was created because of a problem that people don't know the menu below, so it's a function that you don't need unless the design has been reorganized. If there is a similar problem, it is right to just grow a scroll bar without option. and this option has a problem of confusing the scroll bar in the result window.

i am still seeing alot of places where the auto hide scroll bar code exists, are we removing this option completely? if so i can clean it up.

yeah, You can get rid of it.

@jjw24 jjw24 added this to the 1.9.0 milestone Oct 25, 2021
@onesounds
Copy link
Contributor Author

Isn't the plugin image has a refresh animation because it is fetched again? I will check that once I get to my laptop.

@taooceros seems to have forgotten this question. I think we can just do "Refresh Done" on the notification window.

@jjw24
Copy link
Member

jjw24 commented Oct 25, 2021

@onesounds you mentioned this in the description, can you give a bit more context, is it still a problem:

There is a problem of stopping when you close what you opened item. (in plugin page)

@onesounds
Copy link
Contributor Author

the browser bookmark plugin doesnt seem to have the default selection- might be the binding not working?

image

@taooceros seems to have forgotten too.

@onesounds
Copy link
Contributor Author

@onesounds you mentioned this in the description, can you give a bit more context, is it still a problem:

There is a problem of stopping when you close what you opened item. (in plugin page)

I made a mistake in binding. @taooceros fix it at the beginning,

@jjw24
Copy link
Member

jjw24 commented Oct 25, 2021

@taooceros let me know if you have time to look into those above two, if not i can do them, as long as we dont double up.

@jjw24
Copy link
Member

jjw24 commented Oct 25, 2021

@onesounds you mentioned this in the description, can you give a bit more context, is it still a problem:

There is a problem of stopping when you close what you opened item. (in plugin page)

I made a mistake in binding. @taooceros fix it at the beginning,

oh ok, so not a problem anymore, cool i will remove it from the description.

@onesounds
Copy link
Contributor Author

onesounds commented Oct 25, 2021

Remain Tasks in This PR

@taooceros
Copy link
Member

Hm I think I remember I check the refresh button will trigger the image refresh? I forget to reply? 🤣🤣
Anyway, I think adding a notification is reasonable.

@jjw24
Copy link
Member

jjw24 commented Oct 26, 2021

@taooceros really strange with the open in window/tab binding, i dont see any differences between Url and WebSearch plugin views except WebSearch is hooked up to use datacontext but Url refuses to bind for some reason...

@jjw24
Copy link
Member

jjw24 commented Oct 27, 2021

@taooceros really strange with the open in window/tab binding, i dont see any differences between Url and WebSearch plugin views except WebSearch is hooked up to use datacontext but Url refuses to bind for some reason...

Url plugin's selection binding seems to work if i just remove the GroupName attribute, so I may just do that to fix it.

fixes the new tab/window selection binding not working with the new ui
@jjw24 jjw24 enabled auto-merge October 27, 2021 10:31
@jjw24 jjw24 removed the review in progress Indicates that a review is in progress for this PR label Oct 27, 2021
Copy link
Member

@jjw24 jjw24 left a comment

Choose a reason for hiding this comment

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

🥇

@jjw24 jjw24 merged commit f47de6b into Flow-Launcher:dev Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request kind/ui related to UI, icons, themes, etc
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants