8000
Skip to content

Info section UI enhancement#206

Merged
srbhr merged 5 commits intosrbhr:mainfrom
sandipkarmokar:Info_Section_UI_Enhancement
Oct 25, 2023
Merged

Info section UI enhancement#206
srbhr merged 5 commits intosrbhr:mainfrom
sandipkarmokar:Info_Section_UI_Enhancement

Conversation

@sandipkarmokar
Copy link
Copy Markdown
Contributor

Changes Made:

  • added hover states to buttons
  • added similar padding spaces between sub headers
  • added rounded borders to subheader sections and subsequent elements
  • Made some color changes to improver ui/ux(visibility)
  • Other minor fixes

sandip added 5 commits October 20, 2023 10:30
added *pt* to sub headers
added rounded borders to sub sections
added byline highlighting the maximum vector score possible
darkened vector score color to highlight it
added rounded corners to each section
added hover state to buttons
changed border-dotted to dashed
@sandipkarmokar
Copy link
Copy Markdown
Contributor Author

@srbhr @Sayvai please review the PR.

@srbhr
Copy link
Copy Markdown
Owner
srbhr commented Oct 24, 2023

Okay I'll take a look @sandipkarmokar

Copy link
Copy Markdown
Contributor
@Sayvai Sayvai left a comment

Choose a reason for hiding this comment

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

@sandipkarmokar - Nice work Sandi 😎

Managed to run the changes locally, and looks good. No real issues found 🔥

And just as importantly this time, the code changes in the PR are all actually scoped and highlights only the intended changes in the diffs (e.g. without any additional / unnecessary formatting changes), which makes reviewing actual changes easier on the eye for fellow reviewers.

@Sayvai
Copy link
Copy Markdown
Contributor
Sayvai commented Oct 24, 2023

@srbhr Please remember to Squash and Merge this PR to keep the changes to one commit on the upstream main branch 🤓

@srbhr srbhr merged commit f61a9bf into srbhr:main Oct 25, 2023
bmck039 pushed a commit to bmck039/Resume-Matcher that referenced this pull request Jan 27, 2026
* changes made:
added *pt* to sub headers
added rounded borders to sub sections

* Changes made
added byline highlighting the maximum vector score possible
darkened vector score color to highlight it
added rounded corners to each section
added hover state to buttons
changed border-dotted to dashed

* removed spaces from className typo

* removed whitespace from className class

* fixed changes which occured due to formatting issues

---------

Co-authored-by: sandip <sandip@pop-os.localdomain>
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

0