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

Testing Menu Items Issue #998 #2145

Closed
wants to merge 1 commit into from

Conversation

askhellosatya
Copy link

To address this issue, I decided to remove the search form, considering that we already have a search button in the global header area. I've applied a CSS code fix and submitted a patch on GitHub. You can review the changes in this GitHub pull request.

Since I lack experience in development, I'm facing challenges in setting up a testing environment and determining the appropriate reporting channels. I would greatly appreciate any suggestions or assistance in this regard. @bsanevans, I would value your thoughts on this matter.

@jonathanbossenger
Copy link
Collaborator

Hello @askhellosatya, thank you for your pull request. We appreciate your willingness to help get open issues fixed.

As @bsanevans pointed out in this comment, we do need to keep the search bar, so we will need to fix the layout issue without removing the search bar functionality.

I'm going to close this PR for now, but please feel free to continue discussing possible solutions in the original issue, and open a new PR when you find an alternative.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants