Skip to content
This repository has been archived by the owner on Nov 5, 2024. It is now read-only.

Fix alignment of the exclusion string #3084

Merged
merged 1 commit into from
Apr 2, 2024

Conversation

nvllz
Copy link
Contributor

@nvllz nvllz commented Apr 2, 2024

Pull Request (PR) Checklist

Please check if your pull request fulfills the following requirements:

  • The PR is submitted to the main branch.
  • I've read the Contribution Guidelines and my PR doesn't break the rules.
  • I've read and understand the Developer Guidelines.
  • I confirm that I've run the code locally and everything works as expected.
  • 🎬 I've attached a screen recoding of the changes.

What's changed?

Before After
a b

I fixed the alignment of <(excluded)> strings on Accounts, Loans and Transactions screens.

Risk Factors

What may go wrong if we merge your PR?

Not sure how it looks like on smaller screens.

Copy link
Collaborator

@ILIYANGERMANOV ILIYANGERMANOV left a comment

Choose a reason for hiding this comment

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

Looks better, merging after the CI pass

@ILIYANGERMANOV ILIYANGERMANOV merged commit 76d4546 into Ivy-Apps:main Apr 2, 2024
8 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants