Skip to content

Minor style fixes around result rendering#1208

Merged
charisk merged 2 commits intomainfrom
charisk/minor-style-fixes
Mar 15, 2022
Merged

Minor style fixes around result rendering#1208
charisk merged 2 commits intomainfrom
charisk/minor-style-fixes

Conversation

@charisk
Copy link
Contributor

@charisk charisk commented Mar 15, 2022

Fixes for a couple of small styling issues I've noticed.

Checklist

N/A:

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@charisk charisk requested a review from a team as a code owner March 15, 2022 14:23
@charisk charisk force-pushed the charisk/minor-style-fixes branch from 276572f to 617f2ef Compare March 15, 2022 14:26
paddingRight="0.5em"
paddingBottom="0.2em">
paddingBottom="0.2em"
sx={{ wordBreak: 'break-word' }}>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This breaks up a line of code if it's too long

Copy link
Contributor

Choose a reason for hiding this comment

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

But it looked so beautiful! 😆

<VerticalSpace size={1} />
Some repositories have more than {MAX_RAW_RESULTS} results. We will only show you up to {MAX_RAW_RESULTS}
results for each repository.
Some repositories have more than {MAX_RAW_RESULTS} results. We will only show you up to
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This adds a space between MAX_RAW_RESULTS and "results"

Copy link
Contributor

@shati-patel shati-patel left a comment

Choose a reason for hiding this comment

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

Thanks! 💅🏽

@charisk charisk merged commit 46a1dd5 into main Mar 15, 2022
@charisk charisk deleted the charisk/minor-style-fixes branch March 15, 2022 14:43
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