THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added:
FS#997 - awful.widget.graph scroll direction User who did this - Uli Schlachter (psychon) ---------- For patch 1: What exactly do you mean with "is because to get rid of antialias"? According to xmag and a screenshot, my memory usage graph has more than two colors because there is antialiasing at the "edge" of the graph. So I guess that can't be the reason. So what does this patch actually accomplish? (Yes, I do agree that painting the background first does make sense) For patch 2: I'd propose breaking everyone's config again. :-) If left-to-right is easier to implement than right-to-left (your patch looks a little like that), then let's just always force that direction. The other direction could be forced via a layout. Currently there is wibox.layout.rotate, something similar could be done which mirrors a widget. This is likely a little more complicated to use in configs, but is definitely more generic. What do you think about this idea? Alternatively, if you don't like that approach (I'm not really convinced of it myself), why don't you implement the direction via cr:translate(width, 0) and cr:scale(-1, 0)? That way, the actual drawing code wouldn't have to worry about the direction. Sounds easier to me. (However, I'd like this to be implemented in such a way that the pattern (=foreground/background color) is not mirrored) Oh and: If we can mirror a graph in right/left direction, why dont we also mirror it in top/bottom direction? (if you know what I mean) ---------- More information can be found at the following URL: https://awesome.naquadah.org/bugs/index.php?do=details&task_id=997#comment3014 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above. -- To unsubscribe, send mail to [email protected].
