Skip to content

Comments

Show suspend button not lock when actually suspending in greeter#94

Merged
leonardo-lemos merged 4 commits intomainfrom
greeter-lock-not-suspend
Apr 19, 2025
Merged

Show suspend button not lock when actually suspending in greeter#94
leonardo-lemos merged 4 commits intomainfrom
greeter-lock-not-suspend

Conversation

@vjr
Copy link
Member

@vjr vjr commented Mar 22, 2025

Currently in greeter the lock button is displayed but clicking it actually suspends. This PR just shows the suspend button instead to avoid surprises.

Fixes #98

@vjr vjr self-assigned this Mar 22, 2025
@vjr vjr requested a review from a team March 22, 2025 17:06
Copy link
Member

@ryonakano ryonakano left a comment

Choose a reason for hiding this comment

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

It should be always locked when Greeter is shown so it makes sense to me to always show the suspend button and hide the lock button in Greeter. Works as expected too!

@leonardo-lemos leonardo-lemos merged commit d3b4810 into main Apr 19, 2025
4 checks passed
@leonardo-lemos leonardo-lemos deleted the greeter-lock-not-suspend branch April 19, 2025 02:17
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.

Lock button actually suspends in Greeter

3 participants