Bug 76367 - platform/os fields bogus
Summary: platform/os fields bogus
Status: CONFIRMED
Alias: None
Product: bugs.kde.org
Classification: Websites
Component: general (show other bugs)
Version: unspecified
Platform: Unlisted Binaries All
: NOR minor
Target Milestone: ---
Assignee: Matt Rogers
URL:
Keywords:
: 296659 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-02-28 19:41 UTC by Oswald Buddenhagen
Modified: 2022-09-02 19:30 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Oswald Buddenhagen 2004-02-28 19:41:11 UTC
there a too few fields and some of the existing fields are misleading. 
 
Platform should be the hardware platform. that matters for the fewest bugs, so 
maybe it can be left out alltogether. 
 
OS as such is ok, but we need finer granularity for linux distros. 
 
what is now in Platform is not really helpful. this should be called 
"installation source" or something like that and should have only a few 
choices: unspecified/cvs/source tar-ball/official packages/3rd party packages. 
encoding the distribution in the packages makes sense in so far, that 
theoretically it is possible to install redhat rpms on mandrake, etc., but 
possibly it would be better to leave it out and let the reporter mention it 
explicitly if he uses such a setup.
Comment 1 Brad Hards 2004-04-30 13:16:22 UTC
CVS commit by bhards: 

Add support for global kde options --icon and --miniicon, as requested
by bug:76367 for a few of the widgets: textbox, radiolist, checklist,
combobox, menu and password. Also added those to the test script.
Those are the easy ones. Others require a bit more fiddling.

Thanks to David Faure for the guidance on how to address this 
properly.

CCMAIL: 76367@bugs.kde.org


  M +9 -5      test   1.8
  M +8 -0      widgets.cpp   1.7


--- kdebase/kdialog/test  #1.7:1.8
@@ -27,18 +27,22 @@
 
 echo "text box:"
-./kdialog --title "This is a text box" --textbox widgets.h 400 300
+./kdialog --miniicon "about_kde" --title "This is a text box" --textbox widgets.h 400 300
 
 echo "menu:"
-./kdialog --title "This is a menu" --menu "Choose one of these" a English b German c French d Spanish
+./kdialog --miniicon "about_kde"  --title "This is a menu" --menu "Choose one of these" a English b German c French d Spanish
 
 echo "checklist:"
-./kdialog --title "This is a checklist" --checklist "Choose some of these" a English on  b German off  c French off d Spanish on
+./kdialog --miniicon "about_kde" --title "This is a checklist" --checklist "Choose some of these" a English on  b German off  c French off d Spanish on
 
 echo "radiolist:"
-./kdialog --title "This is a radiolist" --radiolist "Choose one of these" a English off b German off c French on d Spanish off
+./kdialog --miniicon "about_kde" --title "This is a radiolist" --radiolist "Choose one of these" a English off b German off c French on d Spanish off
 
 echo "combobox:"
-./kdialog --title "This is a combobox" --combobox "Pick your favorite ice-cream flavor:" "Vanilla" "Chocolate" "Strawberry" "Fudge"
+./kdialog --miniicon "about_kde" --title "This is a combobox" --combobox "Pick your favorite ice-cream flavor:" "Vanilla" "Chocolate" "Strawberry" "Fudge"
 
 echo "passivepopup:"
 ./kdialog --title "This is a passive popup" --passivepopup "It will disappear in about 10 seconds" 10
+
+echo "password:"
+./kdialog --title "This is a password dialog" --icon "desktop" --miniicon "about_kde" --password "Enter the password:"
+

--- kdebase/kdialog/widgets.cpp  #1.6:1.7
@@ -28,4 +28,5 @@
 #include <kcombobox.h>
 #include <kdebug.h>
+#include <kapplication.h>
 
 #include <qlabel.h>
@@ -47,4 +48,5 @@ bool Widgets::passwordBox(QWidget *paren
   KPasswordDialog dlg( KPasswordDialog::Password, false, 0, parent );
 
+  kapp->setTopWidget( &dlg );
   dlg.setCaption(title);
   dlg.setPrompt(text);
@@ -60,4 +62,5 @@ int Widgets::textBox(QWidget *parent, in
   KDialogBase dlg( parent, 0, true, QString::null, KDialogBase::Ok, KDialogBase::Ok );
   dlg.setCaption(title);
+  kapp->setTopWidget( &dlg );
   QTextEdit *edit = new QTextEdit( dlg.makeVBoxMainWidget() );
   edit->setReadOnly(TRUE);
@@ -91,4 +95,5 @@ bool Widgets::comboBox(QWidget *parent, 
                    KDialogBase::Ok );
 
+  kapp->setTopWidget( &dlg );
   QVBox* vbox = dlg.makeVBoxMainWidget();
 
@@ -111,4 +116,5 @@ bool Widgets::listBox(QWidget *parent, c
 
   box.setCaption(title);
+  kapp->setTopWidget( &box );
 
   for (unsigned int i = 0; i+1<args.count(); i += 2) {
@@ -135,4 +141,5 @@ bool Widgets::checkList(QWidget *parent,
 
   box.setCaption(title);
+  kapp->setTopWidget( &box );
 
   for (unsigned int i=0; i+2<args.count(); i += 3) {
@@ -176,4 +183,5 @@ bool Widgets::radioBox(QWidget *parent, 
 
   box.setCaption(title);
+  kapp->setTopWidget( &box );
 
   for (unsigned int i=0; i+2<args.count(); i += 3) {


Comment 2 Brad Hards 2004-04-30 13:35:03 UTC
Damn. All that crud belongs to bug:76237.
Comment 3 Jure Repinc 2004-09-01 19:01:05 UTC
I agree. Bugzilla should be enhanced to support entering the processor/architecture type. Some types allow you to compile in 32-bits and 64-bits (like AMD64) and maybe this could also be added to options to choose from.
Comment 4 Reinhold Kainhofer 2004-09-01 19:45:33 UTC
I can only second Jure's comment. In KOrganizer we had a crash (bug 84979), that would only happen on amd64, but the bug reporter and other people who confirmed the crash never mentioned amd64 (they were not asked for it, so why should they bother?). Of course, we were unable to reproduce the crash on our i386 machines, since we didn't have all information available. Only comment #10 of the report casually mentioned the amd64 platform for the first time.

Such a situation could be prevented in the future if the bug reporter is also asked about his hardware platform.

Cheers,
Reinhold
Comment 5 Jekyll Wu 2012-07-12 02:46:08 UTC
*** Bug 296659 has been marked as a duplicate of this bug. ***
Comment 6 Christopher Yeleighton 2014-07-20 18:04:43 UTC
(In reply to Oswald Buddenhagen from comment #0)
> there a too few fields and some of the existing fields are misleading. 
>  
> Platform should be the hardware platform. that matters for the fewest bugs,
> so 
> maybe it can be left out alltogether. 

Platform has been renamed to Hardware in the UI, which is a complete misunderstanding.  Hardware=openSUSE?  Thank you very much.
Comment 7 Justin Zobel 2021-03-09 05:49:44 UTC
Thank you for the bug report.

As this report hasn't seen any changes in 5 years or more, we ask if you can please confirm that the issue still persists.

If this bug is no longer persisting or relevant please change the status to resolved.