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

Add bar direction configuration #97

Closed

Conversation

FilippoBonazziSUSE
Copy link
Contributor

@FilippoBonazziSUSE FilippoBonazziSUSE commented Jul 5, 2022

Add a direction configuration parameter which can be set to horizontal or vertical.

Proposed implementation for #44 which has been stuck for a while

Resolves #44

Add a direction configuration parameter which can be set to horizontal
or vertical.
@francma
Copy link
Owner

francma commented Aug 1, 2022

The drawing logic is too complicated for my taste (and it was complicated even before this PR). My plan is to replace it with pixman in the future.

I would suggest to drop any attempts to optimize it for performance and just make it as simple as possible for now. When I wrote that memcpy trick, it was more from a curiosity than actual performance optimization needs.

@francma
Copy link
Owner

francma commented Nov 25, 2022

Closing in favor of #108

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

Successfully merging this pull request may close these issues.

Feature request: add rotation option
2 participants