-
Notifications
You must be signed in to change notification settings - Fork 4.8k
[Bug]: Multitool select/deselect all button not showing correctly #3206
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
Comments
I would like to work on this issue. Could you assign this to me? I would also like to clarify just one question regarding the bug: |
Sorry, I should've mentioned @Frooodle as well |
Hi @JoseQuintas2003. Sorry I missed your comment there. Yes you are spot on, the buttons should appear once the page selection has begun. They should not be visible if page selection is not enabled. Please let me know if you have any more questions. |
Thank you so much for the clarification. I'll begin working on it asap. Have a nice day! |
Hi @JoseQuintas2003, how are you getting on? Let me know if you want any assistance, clarification or just someone to bounce an idea off. |
Hey there @ConnorYoh I believe I have already come up with the solution for this issue, but I did noticeone thing which I think I should discuss here before concluding the development. The question is: If you press the "Page Select" button when there are one or more pages currently selected, these pages remain selected despite Page Selection being turned off. Should all pages be deselected when the Page Selection is turned off or is this behavior intentional? (Screenshot that might more easily demonstrate this below) |
Hi @JoseQuintas2003, great to hear you are having success with the issue! |
Yes, @ConnorYoh, I can handle it in the same PR. Since I'm already done with the issue itself, I'll work this part out now and I'll open the said PR then. Thanks for the help. Have a great day! |
Hey there @ConnorYoh, I have just opened the PR for this issue. As promised, the bug fix is also included in there. |
Thanks @JoseQuintas2003, we will take a look and get back to you, thanks for your hard work. It's greatly appreciated |
Installation Method
None
The Problem
In multitool:
Add enabled/disabled colouring to page select button when pressed/depressed
Version of Stirling-PDF
0.44.2
Last Working Version of Stirling-PDF
No response
Page Where the Problem Occurred
No response
Docker Configuration
Relevant Log Output
Additional Information
No response
Browsers Affected
No response
No Duplicate of the Issue
The text was updated successfully, but these errors were encountered: