Skip to content
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

fixed typos in preference-array.css closing issue #9243 #9270

Merged
merged 1 commit into from
Mar 29, 2021

Conversation

arabellastrange
Copy link

@arabellastrange arabellastrange commented Mar 29, 2021

Signed-off-by: Chadha Degachi C.Degachi@student.tudelft.nl

What it does:

Fixes: #9243
This request fixes three typos in the preference-array.css file in the padding variable name.

How to test

To test this fix, navigate to the application preferences from File -> Settings -> Open Preferences and scroll to the Custom Data section in the CSS completion section. Add a number of values and inspect (with chrome developer tools) the displayed array to verify correct behaviour.

padding-fix

Review checklist

Reminder for reviewers

Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

@arabellastrange thank you for your first contribution 👍

In order to accept the changes, please complete the following:

  • please sign the eclipse contributor agreement (eca) with the same email as your sign-off - it is the reason for the failed CI check
  • after completing the updates make sure to squash commits - in the case you might have multiple.

@vince-fugnitto vince-fugnitto added css Issues related to CSS functionality preferences issues related to preferences labels Mar 29, 2021
@arabellastrange
Copy link
Author

Thank you for the reply! I am working on fixing this at the moment

Signed-off-by: chadha <c.degachi@student.tudelft.nl>
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

@arabellastrange thank you for the updates, we hope its the first of many 👍

@vince-fugnitto vince-fugnitto merged commit 4a3492b into eclipse-theia:master Mar 29, 2021
@vince-fugnitto vince-fugnitto added this to the 1.13.0 milestone Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css Issues related to CSS functionality preferences issues related to preferences
Projects
None yet
Development

Successfully merging this pull request may close these issues.

preferences: typo in styling
2 participants