Skip to content

Conversation

yaira2
Copy link
Member

@yaira2 yaira2 commented Dec 28, 2021

Resolved / Related Issues
Items resolved / related issues by this PR.

  • Closes #issue...
  • Related #issue...

Details of Changes
Add details of changes here.

  • Added new layout to the general tab in the properties window

Validation
How did you test these changes?

  • Built and ran the app
  • Tested the changes for accessibility

Screenshots (optional)
Add screenshots here.
image
image

@yaira2 yaira2 marked this pull request as ready for review December 29, 2021 02:22
@yaira2 yaira2 requested a review from gave92 December 29, 2021 02:42
@yaira2 yaira2 changed the title New tab layout in properties dialog Redesigned general tab in properties window Dec 29, 2021
@gave92 gave92 added ready to merge Pull requests that are approved and ready to merge and removed needs - code review labels Dec 29, 2021
@gave92
Copy link
Member

gave92 commented Dec 29, 2021

Looks good to me, though maybe someone else should try to reproduce the crash when scrolling.

@yaira2 yaira2 merged commit 9d00c81 into main Dec 29, 2021
@yaira2 yaira2 deleted the PropertiesDialog branch December 29, 2021 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Pull requests that are approved and ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants