Skip to content
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

Closed
wants to merge 5 commits into from
Closed

Added code of conduct and contributing btns. #123

wants to merge 5 commits into from

Conversation

nlok5923
Copy link
Member

@nlok5923 nlok5923 commented Sep 22, 2020

Fixes

Description

Added button to take a look at contributing guidelines and code of conduct.

Screenshot Before Change:

Screenshot (1344)

Screenshot After Change:

Screenshot (1343)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Non Functional Requirement

  • Follows the code style of this project.
  • Tests Cover Changes
  • I have performed a self-review of my own code
  • All new and existing tests passed.
  • Documentation

Copy link
Member

@aashutoshrathi aashutoshrathi left a 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?

@nlok5923
Copy link
Member Author

nlok5923 commented Sep 22, 2020

Sir please have a look to the site : https://nlok5923-getmein-web-1.glitch.me/

Portal to get an invite to IIITV Github organization

Copy link
Member

@aashutoshrathi aashutoshrathi left a 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

@nlok5923
Copy link
Member Author

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>
Copy link
Member

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
Copy link
Member

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
Copy link
Member

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;
Copy link
Member

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?

@aashutoshrathi
Copy link
Member

Also, @nlok5923 Your commit messages indicate you are using GitHub Web GUI to edit files.
Please understand that we highly discourage this as it skips many layers of checks, please fork, clone and change files in editor locally, so that you automatically get styling fixes (at least).
cc: @iiitv/project-maintainers

@anshumanv
Copy link
Member

Yep, please write meaningful commit messages/descriptions to indicate what the change does. GUI 👎

@nlok5923
Copy link
Member Author

okay sir sorry for using Web ui i will create a new pull request.

@nlok5923 nlok5923 closed this Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants