ssokolow / quicktile

Adds window-tiling hotkeys to any X11 desktop. (An analogue to WinSplit Revolution for people who don't want to use Compiz Grid)
https://ssokolow.com/quicktile/
GNU General Public License v2.0
861 stars 78 forks source link

Add support for ColumnCount in config file #83

Closed avaly closed 6 years ago

avaly commented 7 years ago

Column count is configurable via the config file.

Don't be alarmed by the commands diff, I simply indented all the commands code and put them inside the create_commands function.

ssokolow commented 7 years ago

This would make all of the commands invisible to API documentation tooling. (See here.)

In fact, ignoring the effects on epydoc is what caused your pull request to fail the Travis-CI check.

My plan for fixing this was to rework how commands are registered.

avaly commented 7 years ago

Ohh, I wasn't aware of this limitation on the documentation tool. I'll leave it to you to find a better way then. Feel free to close this.

ssokolow commented 6 years ago

Sorry for taking so long. If you upgrade to the new QuickTile, it'll add a ColumnCount key to the config file the first time you run the new version.

Be warned, however, that the installation process has changed because I split QuickTile into multiple files to get past a mental block. It's still simple and you have the same choices, but you'll want to re-read the relevant section of the README.