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

suggestion (YT) : Little change in Replace Timestamp Action for user convenience & cosmetic reasons. #2441

Closed
5 tasks done
Aryan-Chauhan opened this issue Oct 19, 2024 · 1 comment
Labels
Suggestion Leave any other suggestions

Comments

@Aryan-Chauhan
Copy link

Aryan-Chauhan commented Oct 19, 2024

Application

YouTube

Suggestion description

When user taps on the timestamp, below flyout panel opens. I'm suggesting that this flyout panel doesn't look good.
j

Instead of the above flyout panel, the below flyout panel should open when user taps on the timestamp.
h

Reason: While in full-screen mode, the current flyout panel isn't centred and covers the full screen. The suggested flyout panel is centered and doesn't cover the full screen. Also, because of it's rounded corners, it looks better.

Acknowledgements

  • This issue is not a duplicate of an existing suggestion.
  • This is not a feature request.
  • I have chosen an appropriate title.
  • All requested information has been provided properly.
  • I have written the title and contents in English.
@Aryan-Chauhan Aryan-Chauhan added the Suggestion Leave any other suggestions label Oct 19, 2024
@inotia00
Copy link
Owner

Loading a litho component requires a command inherited from the parent litho view, and timestamp is not a litho view

So this is structurally impossible, regardless of my will

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

No branches or pull requests

2 participants