Skip to content

added zIndex to popover element#3

Closed
BrancuAlexandru wants to merge 1 commit into
codu-code:developfrom
BrancuAlexandru:develop
Closed

added zIndex to popover element#3
BrancuAlexandru wants to merge 1 commit into
codu-code:developfrom
BrancuAlexandru:develop

Conversation

@BrancuAlexandru
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread components/Nav/Nav.tsx
const Nav: NextPage = () => {
return (
<Popover as="header" className="relative">
<Popover as="header" className="relative" style={{zIndex: 10}}>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

use tailwindcss classes for z-index

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

To second Praveen - https://tailwindcss.com/docs/z-index

z-10 should be enough

Copy link
Copy Markdown
Contributor

@NiallJoeMaher NiallJoeMaher left a comment

Choose a reason for hiding this comment

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

Could you remove the package-lock and yarn.lock?

They shouldn't be changed unless there are package changes and made thousands of line changes.
Screenshot 2022-07-23 at 09 45 23
`

NiallJoeMaher added a commit to NiallJoeMaher/codu that referenced this pull request Jun 8, 2026
…odu-code#3)

- settings: Profile · Notifications · Account tabs replace the stacked mono
  eyebrow sections (tabs stay mounted so RHF state survives); sticky save bar
- profile: drop the gradient banner + About tab — header is avatar + name +
  Top-helper + @handle, a mono meta row (joined/location/website), bio, and the
  user's topics as tags (wired via user.topics); Posts · Achievements only
- auth: cap the split-screen to a centered max-w-[1080px] on ultrawide; "or
  with email" divider no longer wraps

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This pull request was closed.
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