Skip to content
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.

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

Make greeter window insensitive when a model is selected #3029

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

amolenaar
Copy link
Member

@amolenaar amolenaar commented Jan 16, 2024

PR Type

What kind of change does this PR introduce?

  • Bug fix
  • Feature
  • Chore (refactoring, formatting, local variables, other cleanup)
  • Documentation content changes

What is the current behavior?

If a model is big, a user does not get any feedback when it's selected in the greeter.

What is the new behavior?

A small quality of life thing, that provides feedback when a user clicks on a recent model or template.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

A small quality of life thing, that provides feedback when a user
clicks on a recent model or template.
@github-actions github-actions bot added the python Pull requests that update Python code label Jan 16, 2024
@danyeaw
Copy link
Member

danyeaw commented Jan 18, 2024

Hi @amolenaar, I think I am being a bit dense on this one, but I don't see any difference in behavior between this branch and main. Could you please explain more about what I am supposed to see?

@amolenaar
Copy link
Member Author

The change is minimal. With this PR, the greeter window will be made grayed out when you select a model.

I change the priority f the load function. Now it should be more visible that the greeter window is made insensitive.

Copy link
Member

@danyeaw danyeaw left a comment

Choose a reason for hiding this comment

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

Got it, thanks @amolenaar!

@danyeaw danyeaw added chore Maintenance related PR and removed python Pull requests that update Python code labels Jan 19, 2024
@danyeaw danyeaw merged commit f95a077 into main Jan 19, 2024
20 checks passed
@danyeaw danyeaw deleted the greeter-insensitive branch January 19, 2024 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Maintenance related PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants