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

root path/top bar not visible in windows #840

Closed
flxn7 opened this issue Feb 18, 2024 · 10 comments
Closed

root path/top bar not visible in windows #840

flxn7 opened this issue Feb 18, 2024 · 10 comments
Labels
bug Something isn't working

Comments

@flxn7
Copy link

flxn7 commented Feb 18, 2024

Hello canop,

I'm on windows without admin rights, enterprise environment. I have tried the scoop built and your precompiled built. When I fire up broot without flags in powershell, the root path/top bar is not showing up. When I move around the file list, the window blinks sometimes and it shows the root path. When I toggle_trim_root it doesnt show it either/no reaction. If I start broot with -T, same behaviour. Have I missed something? On wezterm I have the same results. I'm vividly using broot on unix at home.

Best regards,
flxn7

2024-02-18 11_56_11-

2024-02-18 12_01_48-

@flxn7 flxn7 added the bug Something isn't working label Feb 18, 2024
@flxn7 flxn7 changed the title root line/top bar not visible in windows root path/top bar not visible in windows Feb 18, 2024
@tangent360
Copy link

For now you can roll back to 1.32.0 from Jan 2nd which is the latest release that does not have the issue.

@Canop
Copy link
Owner

Canop commented Feb 18, 2024

I'll work on that as soon as possible. It's just not easy for me because I don't usually have a Windows computer available.

@chtenb
Copy link

chtenb commented Feb 19, 2024

In case this information is useful, I sometimes notice that broot starts flickering when e.g. resize, which is when the top line is sometimes visible. However, it always stabilizes on a state where the top is line not visible. When you need a tester I can help out.

@Canop
Copy link
Owner

Canop commented Feb 19, 2024

Some hints for courageous people willing to recompile broot:

broot 1.33 came with a new version of crossterm.
It's possible removing this part would help: https://github.com/Canop/broot/blob/main/src/app/app.rs#L861-L867
It's also possible it wouldn't be enough.

@Daverd-GM
Copy link

oh this is the bug I found when testing the NerdFontsIcons Pull request
this is what I found

Good night, I was trying to compile the Pull-Request for the Nerd Fonts Icons and I found a bug on the 1.33 and 1.33.1 tags in my machine which is:

Operative System Infromation
Edición Windows 10 Home
Versión 22H2
Se instaló el ‎13/‎11/‎2022
Compilación del SO 19045.4046
Experiencia Windows Feature Experience Pack 1000.19053.1000.0
Nombre del dispositivo Laptop-Daverd
Procesador Intel(R) Core (TM) i5-1035G1 CPU @ 1.00GHz 1.19 GHz
RAM instalada 12,0 GB (11,8 GB utilizable)
Modelo el sistema Aspire A315-56
Tipo de sistema Sistema operativo de 64 bits, procesador x64

The bug is that every line shows one line above of where they should be the screen was flickering while I was moving with the arrows and the title bar which has the current Directory didn't render.
Here is an image of the difference

|1.32|1.33 onwards|
Screenshot of the difference of the issue showing an image.

the flickering appears to show the for an instance the lines in the correct position but they quickly move up one line, the mouse detects them in the correct position, if I click the '3D Objects' folder in the right the selection would be the working directory and if I click the 'AppData' then the select would be the '3D Objects' folder

@chtenb
Copy link

chtenb commented Feb 22, 2024

Some hints for courageous people willing to recompile broot:

broot 1.33 came with a new version of crossterm. It's possible removing this part would help: main/src/app/app.rs#L861-L867 It's also possible it wouldn't be enough.

@Canop I've tested this, and it seems to solve the issue for me, as well as making broot do much less flickering during resizing.
Tested on windows 11 on wezterm.

@Canop
Copy link
Owner

Canop commented Feb 22, 2024

@chtenb Thanks for this. You spare me trying to boot a Windows computer.

@Canop
Copy link
Owner

Canop commented Feb 22, 2024

@chtenb I pushed this change, it's in main : 8c2af67

@Daverd-GM
Copy link

I compiled the last commit it works wonderful in Windows Terminal with PowerShell

@Canop
Copy link
Owner

Canop commented Feb 25, 2024

I think it's OK in broot 1.34. Reopen if I'm wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants