---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124151/
---
(Updated June 25, 2015, 9:39 a.m.)
Status
--
This change has been ma
plasma-devel-requ...@kde.org schreef op 23-6-2015 om 18:22:
Today's Topics:
1. Re: Review Request 124151: Add a default icon to
notifications popup (Eike Hein)
2. Re: Plasma-devel Digest, Vol 84, Issue 78 (Xen)
3. Re: Review Request 124151: Add a default ic
/ On June 23, 2015, 10:40 a.m., Kai Uwe Broulik wrote:
/>>/ > I actually really like it, perhaps we should make KDialog passive popup
not set an icon by default now
/>>/
/>>/ Philipp A. wrote:
/>>/ i also like the no-icon no-space version. why add something that just
takes up unneces
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124151/#review81712
---
Ship it!
Looks cleaner indeed, thanks.
- Sebastian Kügler
On Tue, Jun 23, 2015 at 5:46 PM, Xen wrote:
>
> >>* On June 23, 2015, 10:40 a.m., Kai Uwe Broulik wrote:
> *>>* > I actually really like it, perhaps we should make KDialog passive
> popup not set an icon by default now
> *>> >>* Philipp A. wrote:
> *>>* i also like the no-icon no-space vers
/ On June 23, 2015, 10:40 a.m., Kai Uwe Broulik wrote:
/>>/ > I actually really like it, perhaps we should make KDialog passive popup
not set an icon by default now
/>>/
/>>/ Philipp A. wrote:
/>>/ i also like the no-icon no-space version. why add something that just
takes up unnec
> On June 23, 2015, 10:40 a.m., Kai Uwe Broulik wrote:
> > I actually really like it, perhaps we should make KDialog passive popup not
> > set an icon by default now
>
> Philipp A. wrote:
> i also like the no-icon no-space version. why add something that just
> takes up unnecessary space a
> On June 23, 2015, 10:40 a.m., Kai Uwe Broulik wrote:
> > I actually really like it, perhaps we should make KDialog passive popup not
> > set an icon by default now
i also like the no-icon no-space version. why add something that just takes up
unnecessary space and has no informational value?
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124151/#review81698
---
I actually really like it, perhaps we should make KDialog pass
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124151/
---
(Updated June 23, 2015, 11:42 a.m.)
Review request for Plasma.
Changes
> On June 22, 2015, 5:54 p.m., Kai Uwe Broulik wrote:
> > what about just freeing the space instead? i'm find with this solution
> > though
>
> Martin Klapetek wrote:
> I didn't do that for two reasons - apps/commands setting no icon would
> just look out of place with their notification.
> On June 22, 2015, 5:54 p.m., Kai Uwe Broulik wrote:
> > what about just freeing the space instead? i'm find with this solution
> > though
>
> Martin Klapetek wrote:
> I didn't do that for two reasons - apps/commands setting no icon would
> just look out of place with their notification.
> On June 22, 2015, 5:54 p.m., Kai Uwe Broulik wrote:
> > what about just freeing the space instead? i'm find with this solution
> > though
>
> Martin Klapetek wrote:
> I didn't do that for two reasons - apps/commands setting no icon would
> just look out of place with their notification.
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124151/#review81676
---
Ship it!
applets/notifications/package/contents/ui/Notificat
> On June 22, 2015, 7:54 p.m., Kai Uwe Broulik wrote:
> > what about just freeing the space instead? i'm find with this solution
> > though
I didn't do that for two reasons - apps/commands setting no icon would just
look out of place with their notification. Second is that the icon plays an
i
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124151/#review81666
---
what about just freeing the space instead? i'm find with this
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124151/
---
Review request for Plasma.
Repository: plasma-workspace
Description
---
17 matches
Mail list logo
