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

Add search #418

Merged
merged 4 commits into from Mar 19, 2021
Merged

Add search #418

merged 4 commits into from Mar 19, 2021

Conversation

stevavoliajvar
Copy link
Contributor

@stevavoliajvar stevavoliajvar commented Jan 19, 2021

As per this I made changes.

It now:

  • opens in new tab
  • no shadow
  • has seperate button
  • has better spacing

@m52go

Stevan added 2 commits December 18, 2020 17:09
Added search field that is connected to bisq.wiki search.
@stevavoliajvar
Copy link
Contributor Author

Is there any update on this ? Will it be merged ?

@m52go
Copy link
Contributor

m52go commented Mar 16, 2021

@stevavoliajvar sorry for the delay -- I had pinged our designer off-line a while ago, but then it slipped through the cracks. There some minor improvements I'd like to make, which I'll push in a few hours, and then merge it. It's much improved overall 👍

- Adjust fonts, spacing, and borders
- Add new icon that better fits aesthetic
- Add dark mode styling
- Add media query to hide on small desktops and show
properly on mobile
@m52go
Copy link
Contributor

m52go commented Mar 16, 2021

Updates are now implemented...will merge soon @stevavoliajvar unless you have any further comments.

Light mode desktop

Screenshot from 2021-03-16 12-39-22

Dark mode desktop

Screenshot from 2021-03-16 12-39-31

Mobile

Screenshot from 2021-03-16 12-39-44

@m52go m52go merged commit afd3eb6 into bisq-network:master Mar 19, 2021
@stevavoliajvar
Copy link
Contributor Author

Cool, glad I helped :D

@m52go m52go mentioned this pull request Jul 28, 2021
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.

None yet

2 participants