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

chore(site): rework esl-popup game demo page using ui playground #2357

Merged
merged 11 commits into from
Jun 27, 2024

Conversation

fshovchko
Copy link
Contributor

Closes: #2345

@fshovchko fshovchko added enhancement Improvements and additions to code site Feature, enhancement or bug for demo site needs review The PR is waiting for review labels Apr 18, 2024
@fshovchko fshovchko requested a review from a team April 18, 2024 13:48
@fshovchko fshovchko self-assigned this Apr 18, 2024
@fshovchko fshovchko requested review from dshovchko, ala-n and julia-murashko and removed request for a team April 18, 2024 13:48
Copy link
Collaborator

@ala-n ala-n left a comment

Choose a reason for hiding this comment

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

Works better but still not ideal.

Scenario 1: garb and move the "rocket" fast to the separate corner, but keep cursor inside the area.

Current result: the "rocket" and cursor shifted from each other
Expected result: the cursor does not change its position relatively to "rocket" until bounds are reached.

Suggestion: Please note that the delta approach is not stable for such cases in general. It's better to translate the total pointer movement to our coordinate system. Please look at the carousel touch plugin or scrollbar for reference.

Scenario 2:
The suggestion bellow, regarding RAF decoration makes things a little bit more unstable even if the popup is not opened.
Expected result: it should be super smooth, the user can have tens of popups and hundreds of triggest, and we should not have any performance issues on one trigger with a closed popup (It could be a popup issue but it looks like it's only demo problem, popup feels itself super fine from consumers side with beta release at that moment)

site/src/esl-popup/esl-d-popup-game.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@ala-n ala-n left a comment

Choose a reason for hiding this comment

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

Btw

site/views/draft/popup-game.njk Show resolved Hide resolved
@dshovchko dshovchko closed this Apr 28, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Apr 28, 2024
@dshovchko dshovchko reopened this Apr 28, 2024
@ala-n
Copy link
Collaborator

ala-n commented Apr 28, 2024

Let's discuss on the nearest ESL meeting

@fshovchko fshovchko removed the needs review The PR is waiting for review label May 22, 2024
@ala-n ala-n added needs review The PR is waiting for review and removed under discussion labels Jun 5, 2024
@fshovchko fshovchko requested a review from ala-n June 5, 2024 11:36
@ala-n ala-n requested review from a team and removed request for a team June 7, 2024 09:22
@ala-n ala-n requested a review from abarmina June 7, 2024 09:22
site/views/draft/popup-game.njk Outdated Show resolved Hide resolved
site/views/draft/popup-game.njk Outdated Show resolved Hide resolved
site/views/draft/popup-game.njk Outdated Show resolved Hide resolved
@fshovchko fshovchko requested a review from ala-n June 19, 2024 11:43
site/views/draft/popup-game.njk Outdated Show resolved Hide resolved
Co-authored-by: ala'n (Alexey Stsefanovich) <astsefanovich@exadel.com>
site/views/draft/popup-game.njk Show resolved Hide resolved
fshovchko and others added 2 commits June 25, 2024 15:30
Co-authored-by: Dmytro Shovchko <d.shovchko@gmail.com>
…re/rework-popup-game

# Conflicts:
#	site/src/localdev.less
@ala-n ala-n merged commit 49f337b into main-beta Jun 27, 2024
6 checks passed
@ala-n ala-n deleted the chore/rework-popup-game branch June 27, 2024 15:02
@ala-n
Copy link
Collaborator

ala-n commented Jul 1, 2024

🎉 This PR is included in version 5.0.0-beta.23 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Improvements and additions to code needs review The PR is waiting for review released on @beta site Feature, enhancement or bug for demo site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants