Skip to content
This repository has been archived by the owner on Oct 8, 2020. It is now read-only.

Add demos to ordered layout demos (#146) #154

Merged
merged 1 commit into from
Jun 30, 2020
Merged

Conversation

anasmi
Copy link
Contributor

@anasmi anasmi commented Jun 30, 2020

  • Adding demos to ordered layout demos

* Adding demos to ordered layout demos
// end-source-example

layout.getStyle().set("border", "1px solid #9E9E9E");
component1.getElement().getStyle().set("height", "fit-content");

Choose a reason for hiding this comment

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

CRITICAL Define a constant instead of duplicating this literal "fit-content" 4 times. rule
CRITICAL Define a constant instead of duplicating this literal "height" 4 times. rule

@@ -292,6 +311,8 @@ private void createHorizontalLayoutWithDefaultAlignment() {
Component component3 = createComponent(3, "#37474F");

layout.add(component1, component2, component3);

add(layout);
// end-source-example

component2.getElement().getStyle().set("fontSize", "24px");

Choose a reason for hiding this comment

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

CRITICAL Define a constant instead of duplicating this literal "fontSize" 3 times. rule

@vaadin-bot
Copy link

SonarQube analysis reported 19 issues

  • CRITICAL 15 critical
  • MAJOR 2 major
  • MINOR 2 minor

Watch the comments in this conversation to review them.

Top 10 extra issues

Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

  1. CRITICAL OrderedLayoutView.java#L95: Define a constant instead of duplicating this literal "150px" 6 times. rule
  2. CRITICAL OrderedLayoutView.java#L96: Define a constant instead of duplicating this literal "1px solid #9E9E9E" 35 times. rule
  3. CRITICAL OrderedLayoutView.java#L96: Define a constant instead of duplicating this literal "border" 35 times. rule
  4. CRITICAL OrderedLayoutView.java#L99: Define a constant instead of duplicating this literal "#78909C" 32 times. rule
  5. CRITICAL OrderedLayoutView.java#L100: Define a constant instead of duplicating this literal "#546E7A" 30 times. rule
  6. CRITICAL OrderedLayoutView.java#L101: Define a constant instead of duplicating this literal "#37474F" 19 times. rule
  7. CRITICAL OrderedLayoutView.java#L111: Define a constant instead of duplicating this literal "FlexLayout" 5 times. rule
  8. CRITICAL OrderedLayoutView.java#L132: Define a constant instead of duplicating this literal "200px" 7 times. rule
  9. CRITICAL OrderedLayoutView.java#L248: Define a constant instead of duplicating this literal "padding" 3 times. rule
  10. CRITICAL OrderedLayoutView.java#L255: Define a constant instead of duplicating this literal "HorizontalLayout" 15 times. rule

@tmattsso tmattsso merged commit 141b449 into master Jun 30, 2020
@tmattsso tmattsso deleted the master-cherryPicks branch June 30, 2020 10:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants