Skip to content

added zindex for selected event#147

Closed
vips11 wants to merge 1 commit intomainfrom
vipasha/onboarding
Closed

added zindex for selected event#147
vips11 wants to merge 1 commit intomainfrom
vipasha/onboarding

Conversation

@vips11
Copy link

@vips11 vips11 commented Sep 30, 2025

Notion Ticket

NITS

Summary & Review Focus

Added a zindex value for selected events

Testing Instructions

  1. Run the application
  2. Click on any of the events to see the box shadow

Checklist

  • PR title is descriptive and in imperative tense
  • Commit messages are descriptive, atomic, and follow best practices
  • Linter(s) have been run
  • Requested reviews from the PL and relevant team members

@vercel
Copy link

vercel bot commented Sep 30, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
sistema Ready Ready Preview Comment Sep 30, 2025 3:46am

Copy link
Member

@ChinemeremChigbo ChinemeremChigbo left a comment

Choose a reason for hiding this comment

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

This does not address the issue.

As you can see in the image below although the PER absence is selected, its box-shadow is behind T&C. To see this more easily, feel free to increase the shadow opacity from 0.1 to a greater value e.g rgba(0, 0, 0, 0.9).

image

The actual solution might honestly be out of scope / too difficult, but feel free to keep trying and let me know if you need any help!

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