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

Revert header changes to previous state #344

Merged
merged 4 commits into from
Jan 6, 2025
Merged

Conversation

kasya
Copy link
Collaborator

@kasya kasya commented Jan 6, 2025

Reverting Header component changes to previous state (before PR #252 )

@kasya kasya requested a review from harsh3dev January 6, 2025 02:31
@kasya kasya requested a review from arkid15r as a code owner January 6, 2025 02:31
Copy link
Collaborator

@arkid15r arkid15r 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.
I noticed the cursor goes from pointer to arrow after I click on any header link.

@harsh3dev @kasya could you check on your side?

@harsh3dev
Copy link
Collaborator

Looks good. I noticed the cursor goes from pointer to arrow after I click on any header link.

@harsh3dev @kasya could you check on your side?

For me it stays as a pointer for the time I am hovering on the link.

Screen.Recording.2025-01-06.234920.mp4

@arkid15r
Copy link
Collaborator

arkid15r commented Jan 6, 2025

Looks good. I noticed the cursor goes from pointer to arrow after I click on any header link.
@harsh3dev @kasya could you check on your side?

For me it stays as a pointer for the time I am hovering on the link.
Screen.Recording.2025-01-06.234920.mp4

Thanks for checking. I'll create another issue for that if it persists.

@arkid15r arkid15r enabled auto-merge January 6, 2025 18:23
@arkid15r arkid15r added this pull request to the merge queue Jan 6, 2025
Merged via the queue into main with commit 43e81cd Jan 6, 2025
12 checks passed
@arkid15r arkid15r deleted the revert-header-changes branch January 6, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants