<div dir="ltr"><span style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px">Hey Ali!</span><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px"><br></div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px">No worries, sometimes there's just not enough time and too much work...</div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px">I've recently merged your branch to master and fixed some small things.</div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px"><br></div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px">GtkLabels in Gtk3 have a default center alignment, so you have to explicitly set it to left-aligned (I've already done that in git master). I think notifyd didn't take RTL languages into account. (I'm personally not a speaker of a RTL language and tbh I'm never sure and consequently likely not sensitive enough about what users of RTL languages expect to see in UIs.)</div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px"><br></div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px">I can do the change of only using app-notification if you want, should be fairly easy :)</div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px"><br></div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px">The transparency issue is something we can fix as well (if you wanna work on a patch, that's great), but I'd consider it minor for now.<br></div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px"><br></div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px">I've also fixed some other small bugs but I guess other than that xfce4-notifyd is ready for a first Gtk3 release soon.</div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px">Btw I've preliminarily changed the release version to 0.3.0 for now because that is the next version in the scheme that notifyd followed so far, even if that's not consistent with how other apps in Xfce do it. (Generally I also don't mind jumping to 0.4.0.)</div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px"><br></div><div style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px">Cheers</div><div dir="ltr" style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;font-size:13px"><div>Simon</div></div><br><div class="gmail_quote"><div dir="ltr">On Sun, Jul 3, 2016 at 4:59 PM Ali Abdallah <<a href="mailto:aliovx@gmail.com">aliovx@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Simon,<br>
<br>
Sorry for this very very late reply.<br>
<br>
On 05/29/2016 11:21 PM, Simon Steinbeiss wrote:<br>
> Hey Ali,<br>
><br>
> I tried your Gtk3 port and it works practically flawlessly! Awesome<br>
> work on that!<br>
><br>
> Here are some observations though:<br>
>  * the labels (summary and content) are centered instead of<br>
> left-aligned, would be nice to get that fixed (or is it an intended<br>
> design choice?)<br>
<br>
I'll have a look into it. But I don't remember touching the labels of<br>
the summary and contents. BTW, what about RTL Languages?<br>
<br>
>  * the application uses both .osd and .app-notification classes<br>
> (prioritizing .osd) which may contain conflicting styles in themes<br>
> that define both, so e.g. buttons or progressbars might be taken from<br>
> one class and the rest from another. I'd personally favor going for<br>
> only one class (app-notification) instead to avoid this (or at least<br>
> to set this as the primary class)<br>
Okay, I can base the style on app-notification.<br>
<br>
>  * the transparency setting in the settings dialog conflicts with the<br>
> rgba feature of Gtk3 themes (not sure what should take precedence<br>
> there, as a themer I would go with the theme, also for consistency<br>
> with other notifications using the css class/styles, but users may<br>
> have a different view on this)<br>
<br>
I can add a checkbox to use, by default, the rgba theme settings. User<br>
has to uncheck it to use the transparency slider.<br>
><br>
> Cheers<br>
> Simon<br>
<br>
Cheers<br>
<br>
Ali<br>
<br>
</blockquote></div></div>