Skip to content
This repository has been archived by the owner on Apr 8, 2024. It is now read-only.

feat: add possibility to zoom the course workflow #239

Draft
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

Maxsiom
Copy link
Contributor

@Maxsiom Maxsiom commented Jan 8, 2021

Description

I worked on the possibility to zoom the course workflow in the Course Window. First, I thought about manually scaling each single object. Since this needs a lot of rework in every rendering component, a global scaling value, and more work in future GUI components, I decided to find another solution.
Then I thought about GUI scaling. So I found a solution using the GUI.matrix.

zoom

It works pretty well but there are still to problems I did not find a proper solution yet:

  1. The scrollbars do move when zooming. Tried to create manual ones but I did something wrong so it did not work out, unfortunately.
    bug_scrollbars

  2. When zoomed the context menus are positioned wrongly. It uses Event.current.mousePosition. I tried to overwrite it but it did not work out.
    bug_context

I hope you can do something with this draft PR. 👍

Manual

Panning with middle mouse button.
Zooming with CTRL + Mouse Wheel.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Sorry, something went wrong.

JulieAtInnoactive and others added 9 commits December 18, 2020 11:10

Verified

This commit was signed with the committer’s verified signature.
carloe Carlo Eugster
Co-authored-by: Maximilian Hotter <Maxsiom@users.noreply.github.com>
Co-authored-by: Maximilian Hotter <Maxsiom@users.noreply.github.com>
It is now possible to zoom the course workflow with CTRL + MouseWheel in the Course Window.
Base automatically changed from feature/ui-improvements to develop January 15, 2021 11:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants