-
Notifications
You must be signed in to change notification settings - Fork 2
HTML and CSS Tasks - 01 #1
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
base: master
Are you sure you want to change the base?
Conversation
The scaffold has been done as per the instructions, for the first task
task_1/css_task/index.html
Outdated
| <!--Local CSS--> | ||
| <link rel="stylesheet" href="./css/main.css"> | ||
| <!-- Bootstrap CSS --> | ||
| <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/css/bootstrap.min.css" integrity="sha384-ggOyR0iXCbMQv3Xipma34MD+dH/1fQ784/j6cY/iJTQUOhcWr7x9JvoRxT2MZw1T" crossorigin="anonymous"> |
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.
Kill this, don't use any any external libs or frameworks. If you really need to use a CSS reset: https://cssreset.com/scripts/eric-meyer-reset-css/
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.
Task completed
task_1/css_task/index.html
Outdated
| <!-- Bootstrap CSS --> | ||
| <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/css/bootstrap.min.css" integrity="sha384-ggOyR0iXCbMQv3Xipma34MD+dH/1fQ784/j6cY/iJTQUOhcWr7x9JvoRxT2MZw1T" crossorigin="anonymous"> | ||
| <!-- Font Awesome --> | ||
| <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/5.8.2/css/all.min.css"> |
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 as above
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.
Task completed
task_1/css_task/index.html
Outdated
| </head> | ||
| <body> | ||
| <div class="container"> | ||
| <div class="row"> |
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.
Kill all uses of bootstrap and replicate the same look and feel using plain CSS
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.
Task completed
task_1/css_task/css/main.css
Outdated
| transform: translateY(-12px); | ||
| } | ||
| .task-one_form-input-name:invalid { | ||
| border: 1px solid red; |
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.
use rgb or hsl
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.
Usually have been using hex, that's the preferred standard we've been taught. I shall use rgb
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.
Changes done
| .task-one_form-layout { | ||
| border: 1px solid #d3d3d3; | ||
| border-radius: 5px; | ||
| box-shadow: 0 2px 5px rgba(0,0,0,.10), 0 2px 10px rgba(0,0,0,.10); |
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.
why px here and not rem?
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.
Fixed
task_1/css_task/css/main.css
Outdated
| border-radius: 5px; | ||
| box-shadow: 0 2px 5px rgba(0,0,0,.10), 0 2px 10px rgba(0,0,0,.10); | ||
| height: auto; | ||
| margin: 0.75rem 0rem 0.75rem 0rem; |
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.
Where are you setting the base font size to be referenced by rem?
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.
Fixed
task_1/css_task/index.html
Outdated
| </div> | ||
| <div class="col-12 col-sm-12 col-md-6 col-lg-6 col-xl-6"> | ||
| <label for="lastName">Last Name <span><i class="fas fa-asterisk task-one_form-required"></i></span></label> | ||
| <input type="text" class="form-control task-one_form-input-name" id="user_name" placeholder="enter your last name" pattern="[a-zA-Z ][a-zA-Z ]+"> |
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.
How is "pattern" being triggered?
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.
Here pattern takes in regex. I have used the basic that I know of.
| @@ -0,0 +1,66 @@ | |||
| <!DOCTYPE html> | |||
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.
Why different files? Why not have two different commits to make it easier?
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.
Made two separate commits the second
| @@ -0,0 +1,36 @@ | |||
| .task-one_form-layout { | |||
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.
What is the significance of the naming convention you are using?
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.
It is called Block Element Modifier (BEM), usually start off with the app name, then after the underscore it's the block followed by the element and the modifier that is usually targeted.
A coding standard followed
The scaffold has been done as per the instructions, for the first task