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): add share example #2748

Open
wants to merge 2 commits into
base: main-beta
Choose a base branch
from
Open

chore(site): add share example #2748

wants to merge 2 commits into from

Conversation

yadamskaya
Copy link
Collaborator

ESL Share component has been migrated to UI Playground
@exadel-inc/esl-core-team please let me know if any additional component usage examples I need to add
Closes: #2154

@yadamskaya yadamskaya added site Feature, enhancement or bug for demo site needs review The PR is waiting for review labels Nov 6, 2024
@yadamskaya yadamskaya requested a review from a team November 6, 2024 14:37
@yadamskaya yadamskaya self-assigned this Nov 6, 2024
@yadamskaya yadamskaya requested review from dshovchko, ala-n and fshovchko and removed request for a team November 6, 2024 14:37
Copy link
Contributor

github-actions bot commented Nov 6, 2024

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Copy link

codeclimate bot commented Nov 6, 2024

Code Climate has analyzed commit f08c5a6 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 65.3% (0.0% change).

View more on Code Climate.

@yadamskaya yadamskaya requested a review from a team November 6, 2024 14:39
@ala-n ala-n changed the base branch from main to epic/site-5.0.0 November 7, 2024 12:27
@ala-n ala-n changed the base branch from epic/site-5.0.0 to main-beta November 7, 2024 13:38
@ala-n ala-n changed the base branch from main-beta to epic/site-5.0.0 November 7, 2024 13:38
site/views/examples/share.njk Show resolved Hide resolved
<p>Example of share button with default icon rendering.</p>
<esl-share-button name="linkedin" default-icon></esl-share-button>
<hr>
<script type="text/html"
Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe we can merge this with the previous tab? call it "Share button" and add the example without the icon that was there before? leave all 4 examples

image

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@abarmina I like this idea, thanks

@yadamskaya yadamskaya requested a review from abarmina November 18, 2024 19:06
@ala-n ala-n added not ready for merge and removed needs review The PR is waiting for review labels Dec 2, 2024
Base automatically changed from epic/site-5.0.0 to main-beta December 20, 2024 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not ready for merge site Feature, enhancement or bug for demo site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[🚀site]: migrate esl-share example page to playground base
3 participants