-
Notifications
You must be signed in to change notification settings - Fork 5
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
Ug general commands #265
Ug general commands #265
Conversation
Codecov Report
@@ Coverage Diff @@
## master #265 +/- ##
=========================================
Coverage 37.73% 37.73%
Complexity 493 493
=========================================
Files 149 149
Lines 2862 2862
Branches 323 323
=========================================
Hits 1080 1080
Misses 1715 1715
Partials 67 67
Continue to review full report at Codecov.
|
@@ -10,7 +10,7 @@ | |||
<?import javafx.scene.layout.VBox?> | |||
|
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.
same for this class line 24
@@ -10,7 +10,7 @@ | |||
<?import javafx.scene.layout.VBox?> | |||
|
|||
<?import javafx.scene.layout.Region?> | |||
<BorderPane VBox.vgrow="ALWAYS" id="borderPane" fx:id="borderPane" maxWidth="Infinity" maxHeight="800" minWidth="400" prefWidth="400" xmlns="http://javafx.com/javafx/8.0.172-ea" xmlns:fx="http://javafx.com/fxml/1" styleClass="hairdresser_card"> | |||
<BorderPane VBox.vgrow="ALWAYS" id="borderPane" fx:id="borderPane" maxWidth="Infinity" maxHeight="800" minWidth="300" prefWidth="300" xmlns="http://javafx.com/javafx/8.0.172-ea" xmlns:fx="http://javafx.com/fxml/1" styleClass="hairdresser_card"> | |||
<minHeight> | |||
<Region fx:constant="USE_COMPUTED_SIZE"></Region> | |||
</minHeight> |
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.
same here in line 24
@@ -9,7 +9,7 @@ | |||
<?import javafx.scene.layout.VBox?> | |||
|
|||
<?import javafx.scene.layout.Region?> | |||
<BorderPane id="borderPane" fx:id="borderPane" maxHeight="175" maxWidth="Infinity" minHeight="100" minWidth="300" prefHeight="160" prefWidth="400" xmlns="http://javafx.com/javafx/8.0.172-ea" xmlns:fx="http://javafx.com/fxml/1" styleClass="appt_card"> | |||
<BorderPane id="borderPane" fx:id="borderPane" maxHeight="175" maxWidth="Infinity" minHeight="100" minWidth="300" prefHeight="160" prefWidth="300" xmlns="http://javafx.com/javafx/8.0.172-ea" xmlns:fx="http://javafx.com/fxml/1" styleClass="appt_card"> | |||
<padding> | |||
<Insets bottom="5" left="20" right="5" top="5" /> |
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.
i realised that the string will still overflow when it hits the maximum number of characters and when the scroll bar appears, can you help change the left and right padding to 5 instead?
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.
Do you mean padding instead of margin?
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.
yes! sorry for the typo
LGTM! |
Fixes #253