dua-cli icon indicating copy to clipboard operation
dua-cli copied to clipboard
verified publisher icon Byron

Toggle UI split

Open gabrielfalcao opened this issue 6 months ago • 1 comments

Hi @Byron,

I added a feature to toggle the UI direction because that's useful for long file names in small monitors.

Thanks for making dua-cli!

gabrielfalcao avatar Jun 08 '25 00:06 gabrielfalcao

@Byron I made the changes but am having a hard time understanding why the CI is failing. Would you kindly shed some light there for me?

gabrielfalcao avatar Jun 10 '25 03:06 gabrielfalcao

@Byron Regarding CI, I actually don't think it is broken generally: I've been able to reproduce the failing tests on linux locally by bisecting to the changes introduced in commit 19e1e17 of this PR (I don't have a windows machine to try on but I imagine the culprit is the same). Its not clear to me why these changes would cause the tests to fail though.

joehasson avatar Jun 24 '25 21:06 joehasson

Thanks for debugging CI, that will certainly be useful when trying to fix i! Interestingly, I definitely merge a failed CI, so that must have happened sneakily.

I'd merge this PR without green CI though as CI is unrelated, but am waiting for the other comments to be addressed.

Byron avatar Jun 25 '25 05:06 Byron

Closing due to inactivity. Please feel free to reopen or resubmit. Thanks for your understanding.

Byron avatar Oct 28 '25 03:10 Byron