Show several popups using popperjs - Githubissues

aeternity / superhero-ui

Superhero UI

https://superhero.com

ISC License

19 stars 21 forks source link

Show several popups using popperjs #1139

Closed davidyuk closed 3 years ago

davidyuk commented 3 years ago

depends on #1138

Extracting into separate components:

UserPopup (Popperjs that used in Bootstrap makes it drop up if no space at the bottom, see screenshot)
FeedItemMenu (extracts some logic from TipRecord)
TokenSelect popup (makes to use the same component in 3 places, @onvisions is it a design requirement for them to be different?)
TipInputPopup (extracts some logic from TipInput that is not used in deeplink use case)
CookiesDialog

Screenshot 2021-07-09 at 21 24 51

@Liubov-crypto please ensure that mentioned components work correctly

github-actions[bot] commented 3 years ago

Deployed to feature-refactor-dropdowns.aggregator.z52da5wt.xyz, bundle report

AtanasKrondev commented 3 years ago

There's some weird flicker on the cookies popup while you are scrolling.

flick

davidyuk commented 3 years ago

@AtanasKrondev I've fixed it by disabling a popper plugin that is enabled by default

davidyuk commented 3 years ago

The wallet seems to have a broken dropdown (scroll doesn't work), I see fewer tokens than the dropdown below the post

cc: @etharner

davidyuk commented 3 years ago

I've fixed TokenSelector by adding space between token name and amount