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

fixed button allignment #659 #694

Open
wants to merge 5 commits into
base: gh-pages
Choose a base branch
from
Open

fixed button allignment #659 #694

wants to merge 5 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Apr 29, 2020

align button on center when the screen is small...this is my first commit if anyone notices some mistake please let me know

Copy link
Member

@Chaitanyassr Chaitanyassr left a comment

Choose a reason for hiding this comment

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

Hi @ayush2999 Thanks for the PR. You have to mention the issue number in the PR description and not the PR title. this is so to link the PR with the issue. Also you need to provide deployed link.

index.html Outdated
@@ -527,10 +529,11 @@ <h1>Proudly developed at FOSSASIA</h1>
<h1>Android Apps of FOSSASIA</h1>
</div>
</div>
<div class="row">
<div class="row" >
Copy link
Member

Choose a reason for hiding this comment

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

additional changes.

index.html Outdated

<div class="col-md-4 col-sm-6 col-xs-12 sponsor-column" >
<div class=" text-center" style="margin-bottom: 30px">
<!-- <div class=" text-center" style="margin-bottom: 30px;"> -->
Copy link
Member

Choose a reason for hiding this comment

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

why commenting this line?

index.html Outdated
<a href="https://github.com/fossasia?utf8=✓&q=pslabhttps://github.com/fossasia?utf8=✓&q=pslab" class="btn" target="_self">PSLab GitHub</a>
<a href="https://fossasia.com" class="btn btn-hollow" target="_self">Buy PSLab</a>
</div>

Copy link
Member

Choose a reason for hiding this comment

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

Please make sure to keep the changes min in your PR for a particular issue.

Copy link
Member

Choose a reason for hiding this comment

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

This line does not do anything so not needed.

index.html Outdated

<div class="col-md-4 col-sm-6 col-xs-12 sponsor-column" >
<div class=" text-center" style="margin-bottom: 30px">
<!-- <div class=" text-center" style="margin-bottom: 30px;"> -->
<div class=" text-center" style="margin-bottom: 30px;">
Copy link
Member

Choose a reason for hiding this comment

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

additional change.

@Chaitanyassr
Copy link
Member

Also there is already a PR for this issue which is inactive but under ongoing review.

@ghost
Copy link
Author

ghost commented Apr 29, 2020 via email

@Chaitanyassr
Copy link
Member

Chaitanyassr commented Apr 29, 2020

Thanks...for highlighting my mistake. How would i get to know that the particular issue had solved abd it's under review? I'll make sure to avoid these mistakes in my future PR's

On Wed 29 Apr, 2020, 6:48 PM CHAITANYA, @.***> wrote: @Chaitanyassr commented on this pull request. ------------------------------ In index.html <#694 (comment)>: > @@ -527,10 +529,11 @@

Proudly developed at FOSSASIA

Android Apps of FOSSASIA

-
+
additional changes. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#694 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALDWE3FKL6HF4MHIMIK3UM3RPASJHANCNFSM4MTSZRHQ .

It will be showing in issue page that a PR has been filed. but it will only show when you link the issue number in the PR description. Go to this link #659 and you will see a PR filed just above your comment.

@Chaitanyassr
Copy link
Member

@ayush2999 although that PR is inactive so i believe that there would be no problem if you do a PR.
So for now on please proceed by doing requested changes to this PR.

@ghost
Copy link
Author

ghost commented Apr 29, 2020

ok..sure

@ghost ghost closed this Apr 30, 2020
@ghost ghost reopened this Apr 30, 2020
@ghost
Copy link
Author

ghost commented Apr 30, 2020

issue: #659
Deployed link: https://angry-stonebraker-552ea7.netlify.app/
screenshot after commit:
fossasia

@ghost
Copy link
Author

ghost commented Apr 30, 2020

Sorry it was(custom.css) deleted by mistake, i m trying to delete that commit:6079a30908afbaca0e7157f315c1a2412450eeab
do you know how can i restore it..

@Chaitanyassr
Copy link
Member

Sorry it was(custom.css) deleted by mistake, i m trying to delete that commit:6079a30908afbaca0e7157f315c1a2412450eeab
do you know how can i restore it..

use reset command.

@mariobehling
Copy link
Member

Please resolve conflicts.

Copy link
Member

@Raunaksingh100 Raunaksingh100 left a comment

Choose a reason for hiding this comment

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

PLz restore the custom.css file

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