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

Undo redo buttons disabled state #555

Merged

Conversation

wasky
Copy link
Contributor

@wasky wasky commented May 11, 2024

Undo and redo buttons are now disabled if there is no undo or redo action available.

Copy link

github-actions bot commented Jul 3, 2024

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Jul 3, 2024
@wasky
Copy link
Contributor Author

wasky commented Jul 3, 2024

@burhanrashid52 Do you remember about this PR? :)

@@ -192,6 +192,9 @@ internal class PhotoEditorImpl @SuppressLint("ClickableViewAccessibility") const
return mGraphicManager.redoView()
}

override val redoStackCount: Int
Copy link
Owner

Choose a reason for hiding this comment

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

If we change this to a boolean then the logic moves here :

 override val isRedoAvailable
      get() = mGraphicManager.redoViewsCount > 0

We need to add similar function for undo as well

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@burhanrashid52
Copy link
Owner

@burhanrashid52 Do you remember about this PR? :)

Nop. It just got slipped 😆. Added review comments.

@burhanrashid52
Copy link
Owner

@wasky I am planning to create a new 3.1.0 release in with some dependencies bump. If you can address the review comment than I can include this in the upcoming release together.

@wasky wasky requested a review from burhanrashid52 July 17, 2024 13:19
@burhanrashid52 burhanrashid52 merged commit fd78586 into burhanrashid52:master Jul 23, 2024
3 checks passed
@wasky wasky deleted the undo-redo-buttons-disabled-state branch July 26, 2024 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants