| Summary: | source context not shown - usability bug? | ||
|---|---|---|---|
| Product: | [Unmaintained] kbabel | Reporter: | Manuel Fierlbeck <manuel.fierlbeck> |
| Component: | general | Assignee: | Stanislav Visnovsky <visnovsky> |
| Status: | RESOLVED UNMAINTAINED | ||
| Severity: | wishlist | CC: | cfeck, esigra, nicolasg |
| Priority: | NOR | ||
| Version First Reported In: | 1.9 | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
Manuel Fierlbeck
2004-08-06 18:01:45 UTC
Do you have any idea how to ease the configuration? The current one is aimed at flexibility - because sources can be structured very differently. Hi, Use KURLRequester class as a directory chooser for "Base folder for source code". The KURLRequester mode should be KFile::Directory|KFile::ExistingOnly. Currently KBabel does not check if the typed directory actually exists. SVN commit 465925 by goutte:
Use a URL requester instead of a simple line edit widget for the
base path of the source code
CCBUG:86713
(This does not close the bug, as the bug reporter wnats some more help,
but we are in message freeze.)
M +7 -4 projectprefwidgets.cpp
M +2 -1 projectprefwidgets.h
--- branches/KDE/3.5/kdesdk/kbabel/commonui/projectprefwidgets.cpp #465924:465925
@@ -890,7 +890,7 @@
QHBox* hbox = new QHBox(box);
hbox->setSpacing(KDialog::spacingHint());
- KFile::Mode mode = static_cast<KFile::Mode>( KFile::Directory | KFile::ExistingOnly | KFile::LocalOnly );
+ const KFile::Mode mode = static_cast<KFile::Mode>( KFile::Directory | KFile::ExistingOnly | KFile::LocalOnly );
_poDirEdit = new KURLRequester(hbox, "kcfg_PoBaseDir");
@@ -1141,10 +1141,13 @@
box->setSpacing(KDialog::spacingHint());
QLabel* tempLabel=new QLabel(i18n("&Base folder for source code:"),box);
- _coderootEdit = new QLineEdit(box, "kcfg_CodeRoot");
- tempLabel->setBuddy(_coderootEdit);
+ const KFile::Mode mode = static_cast<KFile::Mode>( KFile::Directory | KFile::ExistingOnly | KFile::LocalOnly );
+ _coderootEdit = new KURLRequester ( box, "kcfg_CodeRoot" );
+ _coderootEdit->setMode( mode );
+ _coderootEdit->setMinimumSize( 250, _coderootEdit->sizeHint().height() );
+ tempLabel->setBuddy( _coderootEdit );
layout->addWidget(box);
-
+
// FIXME: use KConfigXT
_pathsEditor = new KListEditor(page);
_pathsEditor->setTitle(i18n("Path Patterns"));
--- branches/KDE/3.5/kdesdk/kbabel/commonui/projectprefwidgets.h #465924:465925
@@ -58,6 +58,7 @@
class KListEditor;
class ToolSelectionWidget;
class KIntNumInput;
+class KURLRequester;
struct ModuleInfo;
@@ -276,7 +277,7 @@
void itemsChanged ();
private:
- QLineEdit* _coderootEdit;
+ KURLRequester* _coderootEdit;
KListEditor* _pathsEditor;
};
Manuel, try setting the base source folder to: /home/manuel/entw/orthanc/ and add "@CODEROOT@/src/" to the list of "Path Patterns" - actually it should be enough if it's the only one. But not that you lose the only "documentation" there by removing the other @KEYWORDS@.. ;) HTH. I have fixed a few problems with the source references, including the bugs #114041 and #116393. As your project seems to be GNU-like, you could add following rule (however only in the current development version, which will be KBabel 1.11.1 of KDE 3.5.1): @POFILEDIR@/../@COMMENTPATH@ (This new rule is now part of the default rules for new projects.) Have a nice day! On Monday 26 December 2005 11:50, Nicolas Goutte wrote: (...) > POFILEDIR / / COMMENTPATH@ As the KDE Bugs' censor has passed here, please read: (at)POFILEDIR(at)/(dot)(dot)/(at)COMMENTPATH(at) > > (This new rule is now part of the default rules for new projects.) > > Have a nice day! See also bug #85885 which asks for a full documentation of the "source" configuration dialog. For older KBabel versions, you can use: (at)PACKAGEDIR(at)/(dot)(dot)/(at)COMMENTPATH(at) Ad for a GNU(-like) project, you cannot use the catalog manager, it will not make a difference. Meanwhile, the documentation has been updated. (It will be released with KDE 3.5.1.) Have a nice day! KBabel is no longer maintained, please use the KDE 4 translator's tool called "Lokalize" instead. For more information, please visit http://userbase.kde.org/Lokalize If this is a request for a feature which is also missing in Lokalize, please add a comment so that I can reassign the request to the Lokalize authors. You could also file a new request for Lokalize. |