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

Progress bar "overlapping" track length #887

Open
MCT32 opened this issue Sep 17, 2021 · 3 comments
Open

Progress bar "overlapping" track length #887

MCT32 opened this issue Sep 17, 2021 · 3 comments

Comments

@MCT32
Copy link

MCT32 commented Sep 17, 2021

I've noticed since I've updated the track length indicator no longer changes its color when intercepting the progress bar. I used the word overlapping in the title to try to explain it.
image

@Takheer
Copy link

Takheer commented Sep 17, 2021

I also have this issue after pausing and resuming a track:
Screenshot 2021-09-17 at 09 37 41

@sputnick1124
Copy link
Contributor

I'm seeing this as well. It appears that this has occurred due to an upgrade in the version of tui-rs from version 0.14.0 -> 0.16.0 (1ef37e7). I just tested downgrading back to 0.14.0 and the text is properly inverted when overtaken by the gauge bar.

I think we ought to open an issue in tui-rs about this (if one does not already exist) and then maybe roll back the version until it gets fixed, unless we need 0.16.0 for some feature.

@sputnick1124
Copy link
Contributor

sputnick1124 commented Oct 5, 2021

I opened an issue on the tui-rs repo (fdehau/tui-rs#538) describing the regression.

As a stop gap measure for spotify-tui, instead of reverting to an earlier version of tui-rs, we can simply ignore the user's configured progress bar label color until this regression gets fixed. Essentially, we can change src/ui/mod.rs:1052 from

Style::default().fg(app.user_config.theme.playbar_progress_text),

to be

Style::default(), // .fg(app.user_config.theme.playbar_progress_text),  --- To be added back in when fdehau/tui-rs#538 is resolved 

instead. This will make the label default to the progress bar foreground color and it will be properly inverted when the progress overtakes the label.

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