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

[Catalog] Text and Modal chip alignment colour fixed #965

Closed
wants to merge 27 commits into from

Conversation

MrBehl06
Copy link
Contributor

@MrBehl06 MrBehl06 commented Jan 13, 2023

Signed-off-by: Sandeep Behl sandeepbhel05@gmail.com

Description

Current State
Screenshot 2023-01-13 at 14 06 03
Screenshot 2023-01-13 at 13 55 39

Desired State :
Screenshot 2023-01-13 at 13 58 45
Screenshot 2023-01-13 at 14 00 55

This PR fixes #
These are the following issues fixed in this PR:
1)Text colour and aligned fixed of the modal name.
2)Modal-pattern-image aligned with the text.
3)Modal-chip aligned and colour fixed.
4)em, strong, span colours set according to night mode and dark mode

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@netlify
Copy link

netlify bot commented Jan 13, 2023

Deploy Preview for mesheryio-preview ready!

Name Link
🔨 Latest commit 1706e8d
🔍 Latest deploy log https://app.netlify.com/sites/mesheryio-preview/deploys/63ea5ed3d98743000765ac5b
😎 Deploy Preview https://deploy-preview-965--mesheryio-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@MrBehl06
Copy link
Contributor Author

@leecalcote @GaganpreetKaurKalsi Please review the changes.

Copy link
Contributor

@GaganpreetKaurKalsi GaganpreetKaurKalsi left a comment

Choose a reason for hiding this comment

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

Looks good 👍 Great job 💯

@MrBehl06
Copy link
Contributor Author

@leecalcote Please review it.

@MrBehl06 MrBehl06 force-pushed the TextColor branch 2 times, most recently from 9f4ca73 to 9c4adaa Compare January 19, 2023 06:07
@github-actions github-actions bot added area/docs Improvements or additions to documentation area/website labels Jan 19, 2023
@MrBehl06
Copy link
Contributor Author

@leecalcote @GaganpreetKaurKalsi Please help me why there is an error and test failure in this pr.

@leecalcote
Copy link
Member

@MrBehl06 because one of the commits isn't signed off (follow the instructions provide for a permanent solution. follow these for this PR - https://github.com/meshery/meshery.io/pull/965/checks?check_run_id=10744634801).

Also, this PR has a number of other people's commits that are likely included unintentionally (resync your fork).

@leecalcote
Copy link
Member

A couple of merge conflicts have arisen.

@MrBehl06
Copy link
Contributor Author

Okey @leecalcote thank you.

Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

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

Resync fork, sign commits, eliminate merge conflicts

@MrBehl06 MrBehl06 requested review from leecalcote and removed request for Chadha93 January 25, 2023 08:02
MrBehl06 and others added 13 commits January 25, 2023 14:17
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: Sreya Gadepally <sreya@Sreyas-MacBook-Pro.local>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: Sreya Gadepally <sreyagadepally@gmail.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: l5io <ci@layer5.io>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: l5io <ci@layer5.io>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: l5io <ci@layer5.io>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: Shivam-AfA <shivamsood634@gmail.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: Shivam-AfA <shivamsood634@gmail.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: Shivam-AfA <88040318+Shivam-AfA@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: l5io <ci@layer5.io>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

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

Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
@github-actions github-actions bot removed area/website area/docs Improvements or additions to documentation labels Jan 27, 2023
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
Signed-off-by: SandeepBehl <101118926+MrBehl06@users.noreply.github.com>
@stale
Copy link

stale bot commented Mar 16, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the issue/stale Issue has not had any activity for an extended period of time label Mar 16, 2023
@stale
Copy link

stale bot commented Mar 23, 2023

This issue is being automatically closed due to inactivity. However, you may choose to reopen this issue.

@stale stale bot closed this Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue/stale Issue has not had any activity for an extended period of time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants