Review Request 124005: Add week numbers to calendar - Part 2

[Martin Klapetek]

> On June 4, 2015, 11:59 a.m., Kai Uwe Broulik wrote:
> > applets/digital-clock/package/contents/config/main.xml, line 43
> > <https://git.reviewboard.kde.org/r/124005/diff/1/?file=378756#file378756line43>
> >
> >     Why not make it default?
> >     
> >     Yours looks much more beautiful and tidy than the old 4.x version which always confused me.
> 
> Martin Klapetek wrote:
>     "Simple by default, powerful when needed" ;) It's not a feature everyone wants/needs, but it's there when they do.
> 
> Sebastian Kügler wrote:
>     IMO, it should default to on, without an option. Weeknumbers are a pretty basic calendar feature, and most people will probably not even bother looking at the config dialogue, especially since it has never been there. It's simply a missing feature which we now add, it doesn't need to be optional.
>     
>     It's not "powerful when needed", it's "powerful when the user happens to find the option in the config dialog" this way, it's advertised nowhere that this feature is now available.
> 
> David Edmundson wrote:
>     I've never used them. I'm with Martin.
> 
> Kai Uwe Broulik wrote:
>     I'd say bring in the usability team :)
> 
> David Edmundson wrote:
>     effectively done: https://bugs.kde.org/show_bug.cgi?id=299174#c13
>     comments 13 && 14 are Heiko and Thomas
> 
> Kai Uwe Broulik wrote:
>     Ok then
> 
> Martin Klapetek wrote:
>     > It's not "powerful when needed", it's "powerful when the user happens to find the option in the config dialog"
>     
>     But that's the case with about /all/ our "powerful when needed" features, isn't it? How else would you do "powerful when needed"? And it shouldn't be decoupled from the "Simple by default" part, which this really is.
>     
>     > it's advertised nowhere that this feature is now available.
>     
>     Come on now, we add manymanymany new features every release, portion of which is also off by default. That's what release notes are for.
>     
>     --
>     
>     Anywho, from the aformentioned bug, comment by Thomas Pfeiffer:
>     
>     "A configuration option would definitely be helpful here, because the week number is useless for many people, but essential for many others: In many companies, week numbers are used regularly for time planning, but for people not working in such companies they're indeed pretty much meaningless."
>     
>     One more interesting comment by someone:
>     
>     "I can not understand why you can configure the bejesus out of the clock, but the calendar is a immutable monolithic totem to somebody's preferred format." ...to which I have to really agree.
> 
> Sebastian Kügler wrote:
>     I'm ok with that, I trust our usability people. In my perception, weeknumbers are an essential calendar feature, but apparently they're less used outside of "my world". :)
> 
> Martin Gräßlin wrote:
>     While I have never worked in a company where week numbers matter, it's actually the most important feature I need a calender for. Maybe it's a cultural thing, but week numbers are pretty common in Germany - we even used them in school. So IMHO our usability team is wrong here.

On contrary in Czech republic, many people even don't know that such thing exists at all. In here it's quite useless piece of information (unless in corporate environment).


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124005/#review81163
-----------------------------------------------------------


On June 4, 2015, 11:55 a.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124005/
> -----------------------------------------------------------
> 
> (Updated June 4, 2015, 11:55 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> One of the most requested features for Plasma5. This is the applet's part (basically just the config).
> 
> See https://git.reviewboard.kde.org/r/124004/ for part 1 and screenshot.
> 
> Another part will be the (standalone) Calendar applet.
> 
> 
> Diffs
> -----
> 
>   applets/digital-clock/package/contents/config/main.xml 5237160 
>   applets/digital-clock/package/contents/ui/CalendarView.qml b5a080b 
>   applets/digital-clock/package/contents/ui/configAppearance.qml 669b1cc 
> 
> Diff: https://git.reviewboard.kde.org/r/124005/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150604/d4cf8fc0/attachment-0001.html>