Skip to content
This repository was archived by the owner on Jan 19, 2025. It is now read-only.

Conversation

paul0314
Copy link
Contributor

@paul0314 paul0314 commented Jun 27, 2021

Summary of Changes

Fix CSS-Bugs introducted in commit "Move additional duplicate logic to the TreeNode component and use CSS modules where applicable"

  1. select entire line again
  2. make scrollbar from TreeView fullscreen again (1rem margin was missing at top / bottom)
  3. remove unneccessary margin of tree view (parameterview didn't look symmetrical with it especially the box at the top)

Before

image

After

image

paul0314 added 2 commits June 27, 2021 14:49
(fix selection bug introduced in commit "Move additional duplicate logic to the TreeNode component and use CSS modules where applicable")
@paul0314 paul0314 requested a review from lars-reimann June 27, 2021 13:00
Copy link
Member

@lars-reimann lars-reimann left a comment

Choose a reason for hiding this comment

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

LGTM.

@lars-reimann lars-reimann merged commit 1b89c09 into master Jul 2, 2021
@lars-reimann lars-reimann deleted the SelectEntireLineAgain branch July 2, 2021 07:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants