Skip to content

Conversation

@mmisol
Copy link
Collaborator

@mmisol mmisol commented Sep 9, 2020

Purpose

Removes the 500 units bound on the extension panel width. The panel can
be enlarged as much as needed, while respecting the minimum size of the
canvas panel, which remains as 200 units.

Also, the extension panel size is remembered during a session. Serves
both as a nice to have and a avoids a splitter bug.

Also, the workspace references extension is fixed so that it adapts to
bigger sizes. It didn't stretch beyond 600 units.

Declarations

Check these if you believe they are true

  • The codebase is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated

Reviewers

@QilongTang @reddyashish

FYIs

@DynamoDS/dynamo

Removes the 500 units bound on the extension panel width. The panel can
be enlarged as much as needed, while respecting the minimum size of the
canvas panel, which remains as 200 units.

Also, the extension panel size is remembered during a session. Serves
both as a nice to have and a avoids a splitter bug.

Also, the workspace references extension is fixed so that it adapts to
bigger sizes. It didn't stretch beyond 600 units.
@mmisol
Copy link
Collaborator Author

mmisol commented Sep 9, 2020

Workspace references extension looks like this when stretched:

workspace_references_stretched

@QilongTang
Copy link
Contributor

@mmisol Can you install TuneUp and provide a screen shot as well? I suspect changes in the other repo might be needed as well.

@mmisol
Copy link
Collaborator Author

mmisol commented Sep 9, 2020

@QilongTang TuneUp seems to stretch just fine. Do you know what's up with those failures? They don't seem related to my changes and locally the tests pass just fine...

Screen Shot 2020-09-09 at 4 24 25 PM

@QilongTang
Copy link
Contributor

@mmisol I noticed some random failures in @aparajit-pratap 's PR as well, thinking we can ignore those for now

Copy link
Contributor

@QilongTang QilongTang left a comment

Choose a reason for hiding this comment

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

Looks good to me. The change is a bit simpler than I imagined which is cool

@QilongTang QilongTang merged commit fe1d34b into DynamoDS:master Sep 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants