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

fix: dropdown menu handles clicking outside #353

Merged
merged 5 commits into from
Sep 5, 2024

Conversation

ujjwal105
Copy link
Contributor

Description
When tapping anywhere outside the dropdown menu after it has been opened, the dropdown should close.

Related issue(s)
Fixes #352

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

Copy link

netlify bot commented Aug 6, 2024

Deploy Preview for peaceful-ramanujan-288045 ready!

Name Link
🔨 Latest commit 794f746
🔍 Latest deploy log https://app.netlify.com/sites/peaceful-ramanujan-288045/deploys/66d9ade167502300088058cd
😎 Deploy Preview https://deploy-preview-353--peaceful-ramanujan-288045.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 configuration.

Copy link
Member

@Mayaleeeee Mayaleeeee left a comment

Choose a reason for hiding this comment

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

I like the suggestion @ujjwal105
Thanks for catching that.

Copy link
Member

@AceTheCreator AceTheCreator left a comment

Choose a reason for hiding this comment

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

LGTM!!!

@AceTheCreator AceTheCreator merged commit 5a94547 into asyncapi:master Sep 5, 2024
12 checks passed
ashmit-coder pushed a commit to ashmit-coder/conference-website that referenced this pull request Sep 9, 2024
ashmit-coder added a commit that referenced this pull request Nov 16, 2024
* Added cypress - Initial commit

* added landing page test

* added nav link tests

* merge head

* test

* feat: update agenda times (#354)

* feat: add postman logo in financial support section (#357)


Co-authored-by: V Thulisile Sibanda <[email protected]>%0ACo-authored-by: asyncapi-bot <[email protected]>%0ACo-authored-by: prashantrai-30 <[email protected]>

* add-ticket-link (#358)

* feat: update agenda (#361)

* feat: update ticket cards (#359)


Co-authored-by: acethecreator <[email protected]>

* feat: added the CFP form (#348)

* initial commit for CFP and speakers form

* changes to the form

* removed comments

* added example env

* changes to apply btn

* configured the speakers register API

* changed path for CFP

* url correction

* added Online conference in the Navbar

* Made CFP banner and details json dependent

* update cfp page

* Update dates

* Update eventDetails.js

* test commit

* configured to build

* temp changes to form

* changes to form style

* changed email message

* changed the email message

* changed cfp dates

* changes to cfp css

* changes to online page

* deleted useless components

* Update README.md

* Update README.md

* added speakers guideline component

* changes to guidelines

* changed carousel image

* changes for production

* changes to env names

* testing changes

* test 2

* test 3

* test 4

* test 5

* test 6

* changes to netlify.toml

* test 1

* test 2

* test 3

* test 4

---------

Co-authored-by: asyncapi-bot <[email protected]>
Co-authored-by: V Thulisile Sibanda <[email protected]>
Co-authored-by: Ace <[email protected]>
Co-authored-by: acethecreator <[email protected]>

* chore(deps): bump axios from 1.6.0 to 1.7.4 (#368)

* Revert "feat: update ticket cards (#359)"

This reverts commit 066c265.

* revert netlify

* ticket update

* fix-button (#369)

* fix: added online cfp form (#371)

* added online cfp form

* testing different toml configuration

* ehanced config

* testing new config

* fixed typo

* fixed cfp date

* fixed smtp pass typo (#372)

* fix issue #356 (#374)

* Dropdown bug fixed (#353)

Co-authored-by: Cody's Dad <[email protected]>

* feat: update speakers.json (#383)

* Merge master

* package lock changes

* changes to package lock

* fix bug in home

* added navbar mobile tests

* added base venue tests

* added cfp test

* package-lock changes

* added cypress test

* changes to setup versions

* changes to regex

* test commit

---------

Co-authored-by: V Thulisile Sibanda <[email protected]>
Co-authored-by: prashantrai-30 <[email protected]>
Co-authored-by: V Thulisile Sibanda <[email protected]>%0ACo-authored-by: asyncapi-bot <[email protected]>%0ACo-authored-by: prashantrai-30 <[email protected]>
Co-authored-by: acethecreator <[email protected]>
Co-authored-by: asyncapi-bot <[email protected]>
Co-authored-by: Ace <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Saurbh Kumar <[email protected]>
Co-authored-by: Ujjwal Tyagi <[email protected]>
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.

[BUG] Dropdown Menu Doesn't Close When Clicking Outside
3 participants