-
Notifications
You must be signed in to change notification settings - Fork 44
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
Added code of conduct and contributing btns. #123
Conversation
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.
Code Looks Good, can you give a static deployment for us to quick check things?
Sir please have a look to the site : https://nlok5923-getmein-web-1.glitch.me/
|
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.
Can we fit both buttons in the same row, also keep in mind to use 100vh and there should be no scroll bar on this app
okay sir working on it. |
views/index.html
Outdated
@@ -97,13 +97,24 @@ | |||
<button class="uk-button uk-margin-top uk-border-pill | |||
uk-button-default" | |||
type="submit" onclick="showToast()">Submit</button> | |||
<br><br> |
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 uk-margin-top
in element below it instead <br/>
<span class="uk-text-success uk-text-meta uk-text-center uk-margin">Also take a look at </span> | ||
<br> | ||
<a href="https://github.com/iiitv/getmein-web/blob/master/CODE_OF_CONDUCT.md"> | ||
<button class="uk-button uk-margin-top uk-border-pill |
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.
You are using uk-button
the wrong way here.
When we have links, we should approach it like this.
<a class="uk-button uk-margin-top uk-border-pill uk-button-default"
href="https://github.com/iiitv/getmein-web/blob/master/CODE_OF_CONDUCT.md"
Code of Conduct
</a>
<br> | ||
</a> | ||
<a href="https://github.com/iiitv/getmein-web/blob/master/CONTRIBUTING.md"> | ||
<button class="uk-button uk-margin-top uk-border-pill |
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, don't use the button
element directly.
@@ -12,7 +12,7 @@ | |||
body { | |||
margin: 25px; | |||
overflow-y: scroll; | |||
position: fixed; | |||
position:fixed; |
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 is space here removed?
Also, @nlok5923 Your commit messages indicate you are using GitHub Web GUI to edit files. |
Yep, please write meaningful commit messages/descriptions to indicate what the change does. GUI 👎 |
okay sir sorry for using Web ui i will create a new pull request. |
Fixes
Description
Added button to take a look at contributing guidelines and code of conduct.
Screenshot Before Change:
Screenshot After Change:
Types of changes
Non Functional Requirement