Skip to content

[Backport] Remove double semicolon from the style sheets.#16809

Merged
magento-engcom-team merged 2 commits intomagento:2.1-developfrom
gelanivishal:2.1-develop-PR-back-15795
Jul 26, 2018
Merged

[Backport] Remove double semicolon from the style sheets.#16809
magento-engcom-team merged 2 commits intomagento:2.1-developfrom
gelanivishal:2.1-develop-PR-back-15795

Conversation

@gelanivishal
Copy link
Copy Markdown
Contributor

Original Pull Request

#15795

Description

Remove double semicolon from the style sheets.

Fixed Issues (if relevant)

N/A

Manual testing scenarios

N/A

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@sidolov
Copy link
Copy Markdown
Contributor

sidolov commented Jul 15, 2018

@gelanivishal , please, make the same changes as an original PR, currently, your PR contains extra changes.

@gelanivishal
Copy link
Copy Markdown
Contributor Author

@sidolov There is coding standard changes. I have just removed extra space and tab issue. Please check it.

Thank you.

@gelanivishal
Copy link
Copy Markdown
Contributor Author

@sidolov Have you checked my message?

Thank you.

@sidolov
Copy link
Copy Markdown
Contributor

sidolov commented Jul 23, 2018

Hi @gelanivishal , please, sort all CSS properties alphabetically according to our LESS coding standards

@gelanivishal
Copy link
Copy Markdown
Contributor Author

@sidolov Is there any online tool?

@sidolov
Copy link
Copy Markdown
Contributor

sidolov commented Jul 23, 2018

@gelanivishal , unfortunately, our team doesn't provide any tools for that case. You may find the tool by yourself or fix issues manually.

@gelanivishal
Copy link
Copy Markdown
Contributor Author

@sidolov Please review the latest commit.

@magento-engcom-team
Copy link
Copy Markdown
Contributor

Hi @gelanivishal. Thank you for your contribution.
We will aim to release these changes as part of 2.1.15.
Please check the release notes for final confirmation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants