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

Remove borders on separators in main nav Issue#5107 #5508

Conversation

andyheko
Copy link
Contributor

@andyheko andyheko commented May 18, 2021

Changed color of vertical border separator of main navigation to transparent on line#651 of bisq.css, so that the navigation would look cleaner and modern on both light mode and dark mode.

Resolves #5107

Light mode:
image

Dark mode:
image

@boring-cyborg
Copy link

boring-cyborg bot commented May 18, 2021

Thanks for opening this pull request!

Please check out our contributor checklist and check if Travis or Codacy found any issues with your PR. Also make sure your commits are signed, and that you applied Bisq's code style and formatting.

A maintainer will add an is:priority label to your PR if it is up for compensation. Please see our Bisq Q1 2020 Update post for more details.

@andyheko andyheko changed the title Remove borders on separators in main nav #5107 Remove borders on separators in main nav Issue#5107 May 18, 2021
ripcurlx
ripcurlx previously approved these changes May 19, 2021
Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

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

utACK - and congrats to your first successful PR! 👍

@ripcurlx
Copy link
Contributor

But you missed to sign your commit.

@ripcurlx ripcurlx added this to the v1.6.5 milestone May 19, 2021
@ripcurlx
Copy link
Contributor

@andyheko Did you see my comment regarding signing your commit? Otherwise the PR is fine 👍

@ripcurlx ripcurlx modified the milestones: v1.6.5, v1.6.6 May 21, 2021
@andyheko
Copy link
Contributor Author

@ripcurlx I have set up a GPG key but still try to figure out how to sign my commit that's in this pull request

@andyheko
Copy link
Contributor Author

sorry for the delay, but I couldn't find instruction for signing commit that's already pushed

@ripcurlx
Copy link
Contributor

ripcurlx commented May 25, 2021

Hi @andyheko ! I would do following to get this PR in a mergeable state.

> git reset HEAD~2 (deletes your last two commits, but keeps your changes in place)

Delete the empty line you've added for testing and create a signed commit.

> git commit -S -m "Remove borders on separators in main nav #5107" -m "change color of border to transparent on line#651 of bisq.css"

As you changed history of your branch you have to force push now
> git push --force andyheko (or origin depending how you configured your remotes)

I hope that works for you.

change color of border to transparent on line#651 of bisq.css
@andyheko andyheko force-pushed the Remove-borders-on-separators-in-main-nav branch from 56fed90 to e1d7da1 Compare May 25, 2021 18:23
@andyheko
Copy link
Contributor Author

@ripcurlx Thanks so much for the help! I think this PR is in a mergeable state now.

Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

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

utACK

@ripcurlx ripcurlx merged commit ca11587 into bisq-network:master May 25, 2021
@boring-cyborg
Copy link

boring-cyborg bot commented May 25, 2021

Awesome work, congrats on your first merged pull request!

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.

Remove borders on separators in main nav
2 participants