[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    Re: [Kde-pim] Re: KOrganizer What's This [2]
From:       Reinhold Kainhofer <reinhold () kainhofer ! com>
Date:       2004-10-17 8:35:58
Message-ID: 200410171035.59312.reinhold () kainhofer ! com
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Hi Antonio,

A few comments regarding the what's this messages:

On Sunday 17 October 2004 02:47, Antonio Salazar wrote:
> @@ -44,6 +45,10 @@ KOEditorAttachments::KOEditorAttachments
>    topLayout->setSpacing( spacing );
>  
>    mAttachments = new QListView( this );
> +  QWhatsThis::add( mAttachments,
> +                  i18n("Displays a list of current files that "
> +                       "have been associated with this event or to-do. "

The attachments are not only about attaching files. It's also about attaching 
web pages (URLs), mail messages (kmail's message uid), or contacts, etc. 
Currently possible types are:
-) kmail:XXXXXXXXX  ... reference to a mail in kmail (created by drag and drop 
of the message to the attachment list or the korganizer icon in kontact)
-) mailto:XXX@XXX.TLD ... Link to send a new mail message
-) uid:XXXXXX ... reference to a contact in kaddressbook (created by dnd on 
the attachment list or the icon in kontact)
-) Any "file" URI, e.g. http://xxxx, ftp://xxxx URLs or local or remote file 
pathes.

I guess this list will even grow in the future (after all, it's just one 
additional if loop in the UriHandler::process method).


>    button = new QPushButton( i18n("&Edit..."), this );
> +  QWhatsThis::add( button,
> +                  i18n("Shows the same dialog as the Add button, but "
> +                       "instead of adding an attachment, it modifies "
> +                       "the one currently selected in the list above.") );

Is it really important that the add and edit dialogs are the same? 


>    // Note that the i18n text below is duplicated in readEvent
> +  QString whatsThis = i18n("Sets the identity corresponding to "
> +                          "the organizer of this to-do or event. "
> +                          "Identities can be set either in the 'Personal'
> " +                          "section of the KOrganizer configuration, or
> in the " +                          "'Security & Privacy'->'Password & User
> Account' " +                          "section of the KDE Control Center.
> If you choose " +                          "to set it globally for KDE in
> the Control Center, " +                          "be sure to check 'Use
> email settings from " +                          "Control Center' in the
> 'Personal' section of the " +                          "KOrganizer
> configuration.");

Actually, the list of organizer values is created from all of the following 
values:
-) All identities in KMail
-) The identity you set in korganizer's config or in the control center
-) All additional email addresses that you enter in korganizer's config
-) The "This is me" entry in the addressbook


> +  QWhatsThis::add( mListView,
> +                  i18n("Displays information about current attendees. "
> +                       "Clicking on a column title will sort the list "
> +                       "according to that column.") );

How about adding "To edit an attendee, select it in this list, and directly 
modify the values in the widgets below. The RSVP column indicates whether a 
response is requested from the attendee."

>    button = new QPushButton( i18n("Reload"), this );
> +  QWhatsThis::add( button,
> +                  i18n("Reloads the Gantt chart and the list of attendees
> " +                       "for this to-do.") );

Actually, this reloads all free/busy data for all attendees from the 
corresponding servers.


> +  QWhatsThis::add( mDescriptionEdit,
> +                  i18n("Sets the description for this event. This "
> +                       "will be displayed in a reminder if one is set, "
> +                       "as well as in a tooltip when you hover over the "
> +                       "event and on the event itself it is long enough.")
> );

Nope, the description is never shown on the item itself, only in the tooltip 
or the "show" dialog (which is basically the sameas the reminder).


> +  QWhatsThis::add( mAlarmSoundButton,
> +                  i18n("Sets a sound to play in conjunction with the popup
> " +                       "dialog as a reminder.") );
>    mAlarmSoundButton->setPixmap(KOGlobals::self()->smallIcon("playsound"));
>    mAlarmSoundButton->setToggleButton(true);
>    QToolTip::add(mAlarmSoundButton, i18n("No sound set"));
> @@ -183,6 +218,9 @@ void KOEditorGeneral::initAlarm(QWidget
>    alarmLayout->addWidget(mAlarmSoundButton);
>  
>    mAlarmProgramButton = new QPushButton(parent);
> +  QWhatsThis::add( mAlarmProgramButton,
> +                  i18n("Sets a program to execute in conjunction with the
> " +                       "popup dialog.") );

Actually, I'm not sure if the popup dialog will be triggered in addition to 
the sound/program, or if just the sound/program will be executed, without the 
popup dialog. Have you checked this?


> +  QWhatsThis::add( mTimeButton,
> +                  i18n("Sets whether or not this to-do has a time "
> +                       "associated with it.") );

How about "Sets whether or not this to-dos start and due dates also have a 
time associated with it."


> +  QString whatsThis = i18n("Sets the priority of this to-do on a scale "
> +                          "from one to nine, with one being the higest "
> +                          "priority and nine being the lowest. "
> +                          "In programs that have a "
> +                          "different scale, the numbers will be adjusted "
> +                          "to match the appropriate scale.");

Priority 5 is always understood as "Medium" priority, maybe we should mention 
this.

> +  QWhatsThis::add( combo,
> +                  i18n("The number of the week from the beginning "
> +                       "of this month on which this event or to-do "
> +                       "should recur.") );

I would use "of the month" instead of "of this month", as the recurrence is 
not restricted to this month.


Cheers, and thanks for the work!
Reinhold

-- 
------------------------------------------------------------------
Reinhold Kainhofer, Vienna, Austria
email: reinhold@kainhofer.com, http://reinhold.kainhofer.com/
 * Financial and Actuarial Mathematics, TU Wien, http://www.fam.tuwien.ac.at
 * K Desktop Environment, http://www.kde.org/, KOrganizer / KPilot maintainer

[Attachment #5 (application/pgp-signature)]

_______________________________________________
kde-pim mailing list
kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
kde-pim home page at http://pim.kde.org/

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic