-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Feature: Added "Properties" button to the Details Pane #12913
Feature: Added "Properties" button to the Details Pane #12913
Conversation
I'd like to get more feedback before changing it |
The proposed design has it on the left, this way it lines up with the other details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Can we change the data for "Horizontal Resolution" and "Vertical Resolution" to be just
EDIT: Please note the use of the Multiplication symbol ×, and not the lowercase x |
I agree, what do you think @yaira2 |
Agreed, let's do that |
Actually, |
Resolved / Related Issues
Closes Feature: Add "Edit properties" button to the Details pane #12796
Validation
How did you test these changes?
Properties
Screenshots (optional)