Skip to content

fix: param settings FAB obstructs StoryContent #68

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

Merged
merged 4 commits into from
May 2, 2025

Conversation

congvc-dev
Copy link
Collaborator

New screenshot:
image

@congvc-dev
Copy link
Collaborator Author

I found that FAB obstruct preview content in bottom left corner. So I moved param settings button to the TopAppBar. Also useEmbeddedView is passed down too deep, so i created LocalIsEmbeddedView

@JSMonk
Copy link
Member

JSMonk commented May 2, 2025

Thank you so much for all of your contributions 🙏

@JSMonk JSMonk merged commit 8a3a310 into Kotlin:main May 2, 2025
2 checks passed
exit = fadeOut(),
modifier = Modifier.align(Alignment.BottomEnd),
) {
SmallFloatingActionButton(
Copy link
Collaborator

@eymar eymar May 14, 2025

Choose a reason for hiding this comment

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

This Floating Action Button was added here according to the re-design by our team's designer. It should've not been removed from here.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sorry to hear that. I'll make a revert PR later

Copy link
Collaborator

@eymar eymar May 14, 2025

Choose a reason for hiding this comment

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

I understand that you didn't know about it.
I'd be glad to share the design files, but unfortunately they are in private Figma, and we can't share it.

I appreciate your contributions and improvements in the project 🙂


I'm currently working on a branch which I'm going to push soon (today). I already added a revert of this PR there. So you don't need to revert yourself.


In the future please consider:

  • updating the versions of the dependencies in seprate PRs (gradle, agp, etc), unless they are absolutely related to the feature/fix you implement.

In this case the new gradle version affect the Compose project where we use Storytale. We can't update the gradle there yet. So for now we'll need to stick to an older gradle version here.

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

Successfully merging this pull request may close these issues.

3 participants