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

[BUG] lyrics full screen button rarely works #171

Closed
aneesh1122 opened this issue Apr 13, 2024 · 16 comments
Closed

[BUG] lyrics full screen button rarely works #171

aneesh1122 opened this issue Apr 13, 2024 · 16 comments

Comments

@aneesh1122
Copy link

I touched the lyrics full screen icon

Expected Behavior

It should have expanded easily

Current Behavior

It doesn't expand even after multiple touches

Steps to Reproduce

1.open any song
2.enable lyrics
3.let the lyrics load
4.click the icon which gets the lyrics in full screen

Screenshots

VID_20240414_001911.mp4

Logs

Possible Solution

@MSOB7YY
Copy link
Member

MSOB7YY commented Apr 14, 2024

that's due to artwork being scaled up, this causes the hittest to slightly go out of range, for more info check #50 (comment)

  • workaround:
    • pinch in/zoom in while synced lyrics is shown to enter fullscreen
  • fix:
    1. enable artwork gestures(scale multiplier) from customization settings
    2. open miniplayer/player ui
    3. pinch out/zoom out to scale artwork down
    4. optional: disable artwork gestures again

@aneesh1122
Copy link
Author

that's due to artwork being scaled up, this causes the hittest to slightly go out of range, for more info check #50 (comment)

  • workaround:

    • pinch in/zoom in while synced lyrics is shown to enter fullscreen
  • fix:

    1. enable artwork gestures(scale multiplier) from customization settings
    2. open miniplayer/player ui
    3. pinch out/zoom out to scale artwork down
    4. optional: disable artwork gestures again

The fix is working although pinch in/zoom in only increases the lyrics font, it doesn't let me enter full screen.

@MSOB7YY
Copy link
Member

MSOB7YY commented Apr 14, 2024

ops ur right lemme check

@NikunjKhangwal
Copy link

Since pinching is now used for scaling text size, the fullscreen method isn't possible. I would recommend changing double tap on artwork to enter fullscreen lyrics directly as a workaround for now. We already have extra button to toggle minimized lyrics anyway. In future more shortucts could be added for tapping, double tap and hold as mentioned in #81

@aneesh1122
Copy link
Author

Since pinching is now used for scaling text size, the fullscreen method isn't possible. I would recommend changing double tap on artwork to enter fullscreen lyrics directly as a workaround for now. We already have extra button to toggle minimized lyrics anyway. In future more shortucts could be added for tapping, double tap and hold as mentioned in #81

I actually like how I can have lyrics by double tapping on the album cover.

The fix works great though. At 90% the full screen toggle works perfectly and the cover is barely smaller

@NikunjKhangwal
Copy link

Just a workaround for time being, not a permanent solution. I like it too :)

@MSOB7YY
Copy link
Member

MSOB7YY commented Apr 16, 2024

@NikunjKhangwal @aneesh1122 i agree that double tapping should enter fullscreen, or at least make it optional
so the proposal:

  • tap: does nothing (used to tap lyric line to jump to it)
  • double tap: could be "enter fullscreen" or "toggle lyrics"
  • long press: could be "enter fullscreen" or "toggle lyrics"
  • pinch/zoom in: scale font size

@NikunjKhangwal
Copy link

How about:
Tap: Trigger minimized lyrics (thumbnail lyrics) when they're not active and keep lyrics seeking when lyrics are shown
Double Tap: Directly enter fullscreen lyrics
Hold: Honestly could be used for anything. Like showing add to playlist pop-up, track info, lyrics options etc. In Musicolet it shows the option to view song artwork freely and replacing artwork with local pictures.

@aneesh1122
Copy link
Author

How about: Tap: Trigger minimized lyrics (thumbnail lyrics) when they're not active and keep lyrics seeking when lyrics are shown Double Tap: Directly enter fullscreen lyrics Hold: Honestly could be used for anything. Like showing add to playlist pop-up, track info, lyrics options etc. In Musicolet it shows the option to view song artwork freely and replacing artwork with local pictures.

how will you close the minilyrics?

@NikunjKhangwal
Copy link

how will you close the minilyrics?

By tapping lyrics button on bottom right. Or if something additional can be implemented then another app i use close minilyrics by tapping on the edges of thumbnail and lyrics seeking happen only when tapping in middle.

@aneesh1122
Copy link
Author

aneesh1122 commented Apr 16, 2024

how will you close the minilyrics?

By tapping lyrics button on bottom right. Or if something additional can be implemented then another app i use close minilyrics by tapping on the edges of thumbnail and lyrics seeking happen only when tapping in middle.

Well, the touching region is limited enough already. I can only touch the middle one and sometimes above and below the middle one.

The lyrics can be closed by touching an empty area. An area where there are no words.

@NikunjKhangwal
Copy link

Well, the touching region is limited enough already. I can only touch the middle one and sometimes above and below the middle one.

The lyrics can be closed by touching an empty area. An area where there are no words.

I'm guessing that would be harder to implement but let see what dev have to say

@MSOB7YY
Copy link
Member

MSOB7YY commented Apr 18, 2024

i suggest removing lyrics from namida

MSOB7YY added a commit that referenced this issue May 19, 2024
@MSOB7YY
Copy link
Member

MSOB7YY commented May 19, 2024

yo just allowed entering fullscreen by long pressing. sadly allowing double tap will make normal tap delayed so i gave up on the whole gesture customizing thing, thats for lyrics.

so the current issue has a fix for the button & separate gesture to enter lyrics fullscreen. as for album artwork gestures, there is another request for that at #81

@aneesh1122
Copy link
Author

yo just allowed entering fullscreen by long pressing. sadly allowing double tap will make normal tap delayed so i gave up on the whole gesture customizing thing, thats for lyrics.

so the current issue has a fix for the button & separate gesture to enter lyrics fullscreen. as for album artwork gestures, there is another request for that at #81

Thanks man

@NikunjKhangwal
Copy link

Cool. Now i can make my artworks bigger freely. Thanks :)

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

No branches or pull requests

3 participants