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

swaybar: fix rendering of border and background #3394

Merged
merged 1 commit into from
Jan 10, 2019

Conversation

RedSoxFan
Copy link
Member

Related to #3329

This fixes the rendering of borders and backgrounds for blocks. This
also makes the following changes:

  • both borders and padding are scaled with the output
  • both lines and rectangles are rendered without an antialiasing to
    avoid bleeding outside the desired area

Follow-up points for the completion of #3329:

  • Make the vertical padding optional
  • Make the edge margin optional

swaybar/render.c Outdated
double x, double y, double width, double height) {
cairo_set_source_u32(cairo, color);
cairo_antialias_t antialias = cairo_get_antialias(cairo);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can use cairo_save instead.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh right. Changed

This fixes the rendering of borders and backgrounds for blocks. This
also makes the following changes:
* both borders and padding are scaled with the output
* both lines and rectangles are rendered without an antialiasing to
avoid bleeding outside the desired area
@emersion emersion merged commit 058428b into swaywm:master Jan 10, 2019
@emersion
Copy link
Member

Thanks!

@RedSoxFan RedSoxFan deleted the bar-block-render branch January 10, 2019 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants