Hello again,
there is a problem with QIndex. According to .../qt2/Dialogs.C, the class and its dialog are used for two things nowadays: Labels and indexes. For that reason, the tooltip in QIndexDialogBase.ui is not longer valid. I propose to remove it (along with the label ("Keyword")) from the ui file.
You could be smart and add code to QIndexDialog.C to add the tooltip in QIndexDialog's constructor depending on the dialog type? You could also simplify the code in Dialogs.C by defining a Type enum rather than by passing all those strings. Eg:
class QIndex {
public:
enum Type {
Index,
Label
}; QIndex(Dialog & parent, Type type)
: base_class(parent,
type == Index ? _("Index Entry") : _("Label")),
type_(type),
label_(type == Index ? qt_("&Keyword") : qt_("&Label"))
{} // Used by QIndexDialog.C's c-tor when setting tooltips, etc.
Type type() const { return type_; }private:
Type type_;
QString const label_;
};IMHO we should also rename the class and its dialog to avoid confusion. Could somebody with some knowledge of the QT frontend and CVS access please do this?
Surely, you have enough info to have a go yourself now?
I am also wondering whether it is possible to merge QIndexDialog and QAskForTextDialog (or remove the latter)...
Slowly, Michael, slowly :)
Angus
