With border auto, height does not consider text

Tomas-M / xlunch

Graphical app launcher for X with minimal dependencies

http://xlunch.org

GNU General Public License v3.0

219 stars 37 forks source link

With border auto, height does not consider text #65

Closed Tomas-M closed 6 years ago

Tomas-M commented 6 years ago

When calculating borders in the auto mode, width is OK, but height is not considering the height of text, so if I put 3 rows of small icons, those rows are overlapping, as can be seen here:

PMunch commented 6 years ago

Turns out it was just an arithmetic error in the consideration of prompt height + spacing. I've fixed it locally and will push it along with some other fixes soon.

PMunch commented 6 years ago

Fixed in https://github.com/Tomas-M/xlunch/commit/4defb28b4ec3460205a8835918b3d8036c69ce7e

Tomas-M commented 6 years ago

Awesome, thank you :)