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

release of v10.60 #4361

Merged
merged 23 commits into from
Dec 10, 2024
Merged

release of v10.60 #4361

merged 23 commits into from
Dec 10, 2024

Conversation

langz
Copy link
Contributor

@langz langz commented Dec 5, 2024

No description provided.

langz and others added 5 commits December 2, 2024 12:19
We forward the props to HeightAnimation, so we just need the types and
the docs to make this work.

Maybe we should call it `onVisible`? But on the other hand, it's meant
to be used only when `animate` is set to true.

---------

Co-authored-by: Anders <[email protected]>
I think we should use `onVisible` for both the animated and non-animated
state change. Here is the related PR #4350

While `onAnimationEnd` should be used to determine if the animated has
"ended". Pretty much like HeightAnimation works.

---------

Co-authored-by: Anders <[email protected]>
Copy link

vercel bot commented Dec 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
eufemia ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 9, 2024 1:20pm

Copy link

codesandbox-ci bot commented Dec 5, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

The reason behind this is that I saw a negative side-effect in PR #4357
when it comes to the initial render, where we don't want an animation,
but it did animate.
- Change `playwright.config.ts` so it runs locally on post 8000.
- Copy fonts folder in /public directory so it can be used by third
party apps without getting hashed.
- Document the hosted fonts on [this
page](https://eufemia-git-feat-font-cdn-eufemia.vercel.app/uilib/typography/)
(could not find a better place, any better ideas out there?).
- Added a e2e test to verify the fonts are located properly.
- Moved `Typography.e2e.spec.ts` from the Portal workspace to the
Eufemia Workspace, because I think it's the correct place.
langz and others added 3 commits December 9, 2024 11:47
This only adds the "sync" onFileClick event.
Not support for async yet.

Not really sure how the loading state of a Value.Upload with async
onFileClick would look 🤔


Can be demoed here:

https://eufemia-git-feat-upload-value-on-file-click-eufemia.vercel.app/uilib/extensions/forms/Value/Upload/demos/#using-onfileclick

---------

Co-authored-by: Tobias Høegh <[email protected]>
Removes the warning:
```
      Eufemia Icon required: A Tertiary Button requires an icon to be WCAG compliant in most cases, because variant tertiary has no underline.
      (Override this warning using icon={false}, or consider using one of the other variants) 
```
@tujoworker tujoworker merged commit 44b3eef into release Dec 10, 2024
9 of 10 checks passed
@tujoworker
Copy link
Member

🎉 This PR is included in version 10.60.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants