Migrate esl-select and esl-select-list to uip
#2843
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Natalie-Smirnova
force-pushed
the
chore/migrate-esl-select/esl-select-list-to-playground
branch
from
f1a6584 to
6cd0682
Compare
Natalie-Smirnova
requested review from
a team,
dshovchko,
fshovchko,
NastaLeo,
ala-n,
yadamskaya and
abarmina
and removed request for
a team
ala-n
requested changes
Dec 24, 2024
There was a problem hiding this comment.
There is 2 additional big problems at that moment:
- The examples are too short in size so dropdown can't be shown correctly
We need to make playground area bigger or consider to use nonisolated scope (yes I was suggesting an isolated type, but due to presentation in convenience we probably should fallback to usual demo) - Usually the full demo (when all aspects of its implimenation) is cool, but seeing the result from user perspective I' afraid to see so much stub logic for displaying form data. Probably I was wrong here as well and we need to hide this presentation script. Plus one point to use non isolated scope (Yep sorry again, that was my suggestion, but I'm see that I was wrong)
PS: due to holidays progress may be set on hold, it is not blocking 5.0.0 release and initial 5.0.0 site updates.
| @@ -3,11 +3,11 @@ layout: content | |||
| title: Forms - Select List | |||
| seoTitle: Custom select list examples based on ESL web components | |||
| name: Select List | |||
| tags: [examples, beta] | |||
| tags: [examples, playground] | |||
There was a problem hiding this comment.
Suggested change
| tags: [examples, playground] | |
| tags: [examples] |
- outdated with main-beta
Comment on lines
+9
to
+10
| components: | ||
| - esl-select-list |
There was a problem hiding this comment.
Incorrect, due to front etter is yml syntax the offsets are important
Before (in JSON):
aside: {
components: ['esl-select-list']
}Now (in JSON):
aside: ''
components: ['esl-select-list']Or to be precise incorrect even by syntax (mix list/map declaration)
| <form class="form bg-white border p-4" style="min-height: 150px"> | ||
| <div class="form-group"> | ||
| <label for="color_field" class="form-label">Choose your favorite colors:</label> | ||
| <esl-select-list class="border" style="display: block"> | ||
| <select esl-select-target |
There was a problem hiding this comment.
We should have at least one example with a long list for performance testing
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes: <!#2158>