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

Hide PageView tools in CalendarView #265

Merged
merged 3 commits into from Dec 4, 2023

Conversation

csidirop
Copy link
Contributor

@csidirop csidirop commented Nov 28, 2023

This PR hides all PageView tools and the vendor logo in the newspager/calendar view. Most of the buttons were deactivated or had no effect without a page (eg. double view, or rotation). Some where active but made no sense (link to original document).

Hide PageView tools in CalendarView:

grafik

Hide PageView tools in CalendarYearView:

cal3

@beatrycze-volk beatrycze-volk added this to the DFG-Viewer 6.0.0 milestone Dec 4, 2023
@beatrycze-volk beatrycze-volk added the 🐛 bug A non-security related bug. label Dec 4, 2023
Copy link
Contributor

@beatrycze-volk beatrycze-volk left a comment

Choose a reason for hiding this comment

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

Please rebase your PR.

Signed-off-by: Christos Sidiropoulos <csidirop@runbox.com>
Signed-off-by: Christos Sidiropoulos <csidirop@runbox.com>
Signed-off-by: Christos Sidiropoulos <csidirop@runbox.com>
Copy link
Contributor

@beatrycze-volk beatrycze-volk left a comment

Choose a reason for hiding this comment

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

Thanks for your changes!

@beatrycze-volk beatrycze-volk merged commit a9f8dc4 into slub:master Dec 4, 2023
5 checks passed
@csidirop csidirop deleted the OptimizedCalendarStyling branch January 10, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug A non-security related bug.
Development

Successfully merging this pull request may close these issues.

None yet

2 participants