Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-08 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/#review92148 --- Kai, Just wanted to say that this looks great :) Thank you

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/ --- (Updated Feb. 7, 2016, 11:10 p.m.) Status -- This change has been

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-05 Thread Marco Martin
> On Feb. 2, 2016, 9:22 a.m., Marco Martin wrote: > > hmm, i don't think i like the back/forward buttons both at the right, it > > looked more nicely symmetric before > > Kai Uwe Broulik wrote: > It also reduces mouse travel :) Placing the "Previous" button left of the > month name would

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-05 Thread Andrew Lake
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/#review92078 --- Ship it! Ship It! - Andrew Lake On Feb. 4, 2016, 11:19

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-04 Thread Kai Uwe Broulik
> On Feb. 2, 2016, 12:37 vorm., Andrew Lake wrote: > > Nice improvement. If the week numbers look ok then +1 from me. I need some inspiration for how to align the week names, though. The more I look at it now the better I like it. I tried vertically centering them but that looks awful. On the

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-04 Thread Kai Uwe Broulik
> On Feb. 4, 2016, 11:24 nachm., Heiko Tietze wrote: > > File Attachment: How about this? #2 - How about this? #2 > > > > > > Bug 299174 - Remove "week number" from system tray calendar for > > improved readability > > > >

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-04 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/ --- (Updated Feb. 4, 2016, 11:19 nachm.) Review request for Plasma and KDE

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-04 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/#review92070 --- Ship it! Ship It! - Andreas Kainz On Feb. 4, 2016,

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-04 Thread Heiko Tietze
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/#review92071 --- File Attachment: How about this? #2 - How about this? #2

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-04 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/#review92073 --- This diff doesn't seem to have the week days row changes? -

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-02 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/#review91926 --- hmm, i don't think i like the back/forward buttons both at

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-02 Thread Kai Uwe Broulik
> On Feb. 2, 2016, 9:22 vorm., Marco Martin wrote: > > hmm, i don't think i like the back/forward buttons both at the right, it > > looked more nicely symmetric before It also reduces mouse travel :) Placing the "Previous" button left of the month name would look strange. Not a huge fan of

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-02 Thread Sebastian Kügler
> On Feb. 2, 2016, 9:22 a.m., Marco Martin wrote: > > hmm, i don't think i like the back/forward buttons both at the right, it > > looked more nicely symmetric before > > Kai Uwe Broulik wrote: > It also reduces mouse travel :) Placing the "Previous" button left of the > month name would

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-01 Thread andreas kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/#review91904 --- can you add an screenshot with the week names? - andreas

Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-01 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/ --- Review request for Plasma and KDE Usability. Bugs: 336124, 348362 and

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-01 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/ --- (Updated Feb. 1, 2016, 11:12 nachm.) Review request for Plasma and KDE

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-01 Thread Andrew Lake
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/#review91907 --- Nice improvement. If the week numbers look ok then +1 from