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

[Design] Improve Jarbas document page legibility #525

Conversation

mauehara
Copy link

What is the purpose of this Pull Request?
Based on some observations in #421 and some of my own, this Pull Request intend to enhance the overall legibility of the document page in Jarbas.

What was done to achieve this purpose?

  • Organised the content using outlines cards
  • Fixed weird spacings
  • Hide "X results found" if not necessary (only 1 result)
  • Used color only for interactive elements
  • Improved typography hierarchy

How to test if it really works?
Just go to a document page :) Got before/after screenshots here too

BEFORE
Screen Shot 2020-04-24 at 17 44 36

AFTER
Screen Shot 2020-04-24 at 17 44 48

Who can help reviewing it?
Everyone :)

ps: This is my first pull request for an open source project ever :)

@mauehara
Copy link
Author

Hello guys! It's been a while that this PR has been opened. Is there something from the PR process I'm missing here? Can I do anything to help the merge? :)

@sergiomario
Copy link
Collaborator

Hi @mauehara.

We haven't tested its implementation yet. It was very important that you sent this message.
Soon we will give you a feedback. Sorry for the delay in analyzing your PR.

Thank you!

@sergiomario
Copy link
Collaborator

sergiomario commented May 27, 2020

@mauehara I tested its implementation. No inconsistencies regarding responsiveness or dependencies were identified.

@sergiomario sergiomario merged commit bdc5057 into okfn-brasil:master May 27, 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.

2 participants