August 2008 - YaST Commits - openSUSE Mailing Lists

[yast-commit] r50507 - /trunk/gtk/src/YGDumbTab.cc
by rpmcruz＠svn.opensuse.org 31 Aug '08

31 Aug '08

Author: rpmcruz Date: Mon Sep 1 00:04:23 2008 New Revision: 50507 URL: http://svn.opensuse.org/viewcvs/yast?rev=50507&view=rev Log: Should haven't remove this line... Modified: trunk/gtk/src/YGDumbTab.cc Modified: trunk/gtk/src/YGDumbTab.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGDumbTab.cc?rev=50507&r… ============================================================================== --- trunk/gtk/src/YGDumbTab.cc (original) +++ trunk/gtk/src/YGDumbTab.cc Mon Sep 1 00:04:23 2008 @@ -47,6 +47,7 @@ YDumbTab::addItem (item); GtkWidget *tab_label, *image = 0, *label; label = gtk_label_new (YGUtils::mapKBAccel (item->label()).c_str()); + gtk_label_set_use_underline (GTK_LABEL (label), TRUE); if (item->hasIconName()) { string path = iconFullPath (item->iconName()); GdkPixbuf *pixbuf = YGUtils::loadPixbuf (path); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r50506 - in /trunk/gtk: ChangeLog src/YGUtils.cc src/ygtkwizard.c
by rpmcruz＠svn.opensuse.org 31 Aug '08

31 Aug '08

Author: rpmcruz Date: Sun Aug 31 03:00:44 2008 New Revision: 50506 URL: http://svn.opensuse.org/viewcvs/yast?rev=50506&view=rev Log: * src/ygtkwizard.c: use the proper image API so the help icon gets aligned next to the label. Modified: trunk/gtk/ChangeLog trunk/gtk/src/YGUtils.cc trunk/gtk/src/ygtkwizard.c Modified: trunk/gtk/ChangeLog URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/ChangeLog?rev=50506&r1=50505… ============================================================================== --- trunk/gtk/ChangeLog (original) +++ trunk/gtk/ChangeLog Sun Aug 31 03:00:44 2008 @@ -1,4 +1,4 @@ -2008-08-20 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> +2008-08-30 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> * src/YGUtils.cc: stock exceptions for the media labels. @@ -6,6 +6,9 @@ * src/YGRadioButton.cc: set border. + * src/ygtkwizard.c: use the proper image API so the help icon gets + aligned next to the label. + 2008-08-29 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> * src/YGLayout.cc: implemented new libyui widget YButtonBox, making Modified: trunk/gtk/src/YGUtils.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGUtils.cc?rev=50506&r1=… ============================================================================== --- trunk/gtk/src/YGUtils.cc (original) +++ trunk/gtk/src/YGUtils.cc Sun Aug 31 03:00:44 2008 @@ -529,16 +529,8 @@ it = stockMap.find (id); if (it != stockMap.end()) { const std::string &stock_id = it->second; - GdkPixbuf *pixbuf; - pixbuf = gtk_widget_render_icon (button, stock_id.c_str(), GTK_ICON_SIZE_BUTTON, NULL); - if (pixbuf) { - GtkWidget *image = gtk_image_new_from_pixbuf (pixbuf); - gtk_button_set_image (GTK_BUTTON (button), image); - g_object_unref (G_OBJECT (pixbuf)); - } - else - g_warning ("yast2-gtk: setStockIcon(): id '%s' exists, but has no pixbuf", - stock_id.c_str()); + GtkWidget *image = gtk_image_new_from_stock (stock_id.c_str(), GTK_ICON_SIZE_BUTTON); + gtk_button_set_image (GTK_BUTTON (button), image); } else gtk_button_set_image (GTK_BUTTON (button), NULL); Modified: trunk/gtk/src/ygtkwizard.c URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/ygtkwizard.c?rev=50506&r… ============================================================================== --- trunk/gtk/src/ygtkwizard.c (original) +++ trunk/gtk/src/ygtkwizard.c Sun Aug 31 03:00:44 2008 @@ -373,15 +373,12 @@ static GtkWidget *create_help_button() { - GtkWidget *button, *box, *image; - box = gtk_hbox_new (FALSE, 6); + GtkWidget *button, *image; image = gtk_image_new_from_stock (GTK_STOCK_HELP, GTK_ICON_SIZE_BUTTON); - gtk_box_pack_start (GTK_BOX (box), image, FALSE, TRUE, 0); - gtk_box_pack_start (GTK_BOX (box), gtk_label_new (_("Help")), TRUE, TRUE, 0); - gtk_widget_show_all (box); button = gtk_toggle_button_new(); + gtk_button_set_label (GTK_BUTTON (button), _("Help")); + gtk_button_set_image (GTK_BUTTON (button), image); gtk_button_set_focus_on_click (GTK_BUTTON (button), FALSE); - gtk_container_add (GTK_CONTAINER (button), box); return button; } -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r50505 - in /trunk/gtk: ChangeLog src/YGPackageSelector.cc
by rpmcruz＠svn.opensuse.org 30 Aug '08

30 Aug '08

Author: rpmcruz Date: Sat Aug 30 19:48:44 2008 New Revision: 50505 URL: http://svn.opensuse.org/viewcvs/yast?rev=50505&view=rev Log: Revert selector border. Modified: trunk/gtk/ChangeLog trunk/gtk/src/YGPackageSelector.cc Modified: trunk/gtk/ChangeLog URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/ChangeLog?rev=50505&r1=50504… ============================================================================== --- trunk/gtk/ChangeLog (original) +++ trunk/gtk/ChangeLog Sat Aug 30 19:48:44 2008 @@ -4,7 +4,7 @@ * src/YGDumbTab.cc: fixed tab shortcuts. - * src/YGRadioButton.cc & src/YGPackageSelector.cc: set border. + * src/YGRadioButton.cc: set border. 2008-08-29 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> Modified: trunk/gtk/src/YGPackageSelector.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGPackageSelector.cc?rev… ============================================================================== --- trunk/gtk/src/YGPackageSelector.cc (original) +++ trunk/gtk/src/YGPackageSelector.cc Sat Aug 30 19:48:44 2008 @@ -2432,6 +2432,7 @@ : YPackageSelector (NULL, mode), YGWidget (this, parent, true, YGTK_TYPE_WIZARD, NULL) { + setBorder (0); GtkWindow *window = YGDialog::currentWindow(); int enlarge_width = MIN (0.95 * YUI::app()->displayWidth(), 650); int enlarge_height = MIN (0.95 * YUI::app()->displayHeight(), 600); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r50504 - in /trunk/gtk: ChangeLog src/YGDumbTab.cc src/YGPackageSelector.cc src/YGRadioButton.cc src/YGWidget.cc
by rpmcruz＠svn.opensuse.org 30 Aug '08

30 Aug '08

Author: rpmcruz Date: Sat Aug 30 19:47:32 2008 New Revision: 50504 URL: http://svn.opensuse.org/viewcvs/yast?rev=50504&view=rev Log: * src/YGUtils.cc: stock exceptions for the media labels. * src/YGDumbTab.cc: fixed tab shortcuts. * src/YGRadioButton.cc & src/YGPackageSelector.cc: set border. Modified: trunk/gtk/ChangeLog trunk/gtk/src/YGDumbTab.cc trunk/gtk/src/YGPackageSelector.cc trunk/gtk/src/YGRadioButton.cc trunk/gtk/src/YGWidget.cc Modified: trunk/gtk/ChangeLog URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/ChangeLog?rev=50504&r1=50503… ============================================================================== --- trunk/gtk/ChangeLog (original) +++ trunk/gtk/ChangeLog Sat Aug 30 19:47:32 2008 @@ -2,6 +2,10 @@ * src/YGUtils.cc: stock exceptions for the media labels. + * src/YGDumbTab.cc: fixed tab shortcuts. + + * src/YGRadioButton.cc & src/YGPackageSelector.cc: set border. + 2008-08-29 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> * src/YGLayout.cc: implemented new libyui widget YButtonBox, making Modified: trunk/gtk/src/YGDumbTab.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGDumbTab.cc?rev=50504&r… ============================================================================== --- trunk/gtk/src/YGDumbTab.cc (original) +++ trunk/gtk/src/YGDumbTab.cc Sat Aug 30 19:47:32 2008 @@ -47,7 +47,6 @@ YDumbTab::addItem (item); GtkWidget *tab_label, *image = 0, *label; label = gtk_label_new (YGUtils::mapKBAccel (item->label()).c_str()); - gtk_label_set_use_underline (GTK_LABEL (label), TRUE); if (item->hasIconName()) { string path = iconFullPath (item->iconName()); GdkPixbuf *pixbuf = YGUtils::loadPixbuf (path); @@ -61,6 +60,9 @@ } else tab_label = label; + gchar *label_id = g_strdup_printf ("label-%d", item->index()); + g_object_set_data (G_OBJECT (getWidget()), label_id, label); + g_free (label_id); gtk_widget_show_all (tab_label); GtkNotebook *notebook = GTK_NOTEBOOK (getWidget()); @@ -129,12 +131,16 @@ virtual void shortcutChanged() { - GtkNotebook *notebook = GTK_NOTEBOOK (getWidget()); for (YItemConstIterator it = itemsBegin(); it != itemsEnd(); it++) { YItem *item = *it; - GtkWidget *child = gtk_notebook_get_nth_page (notebook, item->index()); - std::string label = YGUtils::mapKBAccel (item->label()); - gtk_notebook_set_tab_label_text (notebook, child, label.c_str()); + gchar *label_id = g_strdup_printf ("label-%d", item->index()); + GtkWidget *label; + label = (GtkWidget *) g_object_get_data (G_OBJECT (getWidget()), label_id); + g_free (label_id); + + std::string text = YGUtils::mapKBAccel (item->label()); + gtk_label_set_text (GTK_LABEL (label), text.c_str()); + gtk_label_set_use_underline (GTK_LABEL (label), TRUE); } } Modified: trunk/gtk/src/YGPackageSelector.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGPackageSelector.cc?rev… ============================================================================== --- trunk/gtk/src/YGPackageSelector.cc (original) +++ trunk/gtk/src/YGPackageSelector.cc Sat Aug 30 19:47:32 2008 @@ -2432,7 +2432,6 @@ : YPackageSelector (NULL, mode), YGWidget (this, parent, true, YGTK_TYPE_WIZARD, NULL) { - setBorder (0); GtkWindow *window = YGDialog::currentWindow(); int enlarge_width = MIN (0.95 * YUI::app()->displayWidth(), 650); int enlarge_height = MIN (0.95 * YUI::app()->displayHeight(), 600); Modified: trunk/gtk/src/YGRadioButton.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGRadioButton.cc?rev=505… ============================================================================== --- trunk/gtk/src/YGRadioButton.cc (original) +++ trunk/gtk/src/YGRadioButton.cc Sat Aug 30 19:47:32 2008 @@ -52,7 +52,6 @@ YGWidget (this, parent, true, getCheckRadioButtonType(), NULL) { IMPL - setBorder (0); if (!is_horizontal_box (parent)) setStretchable (YD_HORIZ, true); setLabel (label); @@ -127,7 +126,9 @@ YGRadioButtonGroup(YWidget *parent) : YRadioButtonGroup (NULL), YGWidget (this, parent, true, GTK_TYPE_EVENT_BOX, NULL) - {} + { + setBorder (0); + } YGWIDGET_IMPL_COMMON YGWIDGET_IMPL_CHILD_ADDED (m_widget) @@ -151,7 +152,6 @@ YGWidget (this, parent, true, GTK_TYPE_CHECK_BUTTON, NULL) { IMPL - setBorder (0); if (!is_horizontal_box (parent)) setStretchable (YD_HORIZ, true); setLabel (label); Modified: trunk/gtk/src/YGWidget.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGWidget.cc?rev=50504&r1… ============================================================================== --- trunk/gtk/src/YGWidget.cc (original) +++ trunk/gtk/src/YGWidget.cc Sat Aug 30 19:47:32 2008 @@ -239,13 +239,13 @@ gtk_widget_hide (m_label); else { gtk_widget_show (m_label); -/* + // add a ':' at the end int last = str.length()-1; if (str [last] != ':' && str [last] != '.' && str [last] != ' ') str += ':'; - // set it as upper case +/* // set it as upper case unsigned int first = (str [0] == '_') ? 1 : 0; if (str [first] >= 'a' && str [first] <= 'z') str [first] += 'A' - 'a'; -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r50503 - in /trunk/gtk: ChangeLog src/YGUtils.cc
by rpmcruz＠svn.opensuse.org 30 Aug '08

30 Aug '08

Author: rpmcruz Date: Sat Aug 30 13:28:24 2008 New Revision: 50503 URL: http://svn.opensuse.org/viewcvs/yast?rev=50503&view=rev Log: * src/YGUtils.cc: stock exceptions for the media labels. Modified: trunk/gtk/ChangeLog trunk/gtk/src/YGUtils.cc Modified: trunk/gtk/ChangeLog URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/ChangeLog?rev=50503&r1=50502… ============================================================================== --- trunk/gtk/ChangeLog (original) +++ trunk/gtk/ChangeLog Sat Aug 30 13:28:24 2008 @@ -1,3 +1,7 @@ +2008-08-20 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> + + * src/YGUtils.cc: stock exceptions for the media labels. + 2008-08-29 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> * src/YGLayout.cc: implemented new libyui widget YButtonBox, making Modified: trunk/gtk/src/YGUtils.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGUtils.cc?rev=50503&r1=… ============================================================================== --- trunk/gtk/src/YGUtils.cc (original) +++ trunk/gtk/src/YGUtils.cc Sat Aug 30 13:28:24 2008 @@ -504,8 +504,14 @@ for (GSList *i = list; i; i = i->next) { gchar *id = (gchar *) i->data; GtkStockItem item; - if (gtk_stock_lookup (id, &item)) - stockMap[cutUnderline (item.label)] = id; + if (gtk_stock_lookup (id, &item)) { + const gchar *_id = id; + if (!strcmp (id, GTK_STOCK_MEDIA_NEXT) || !strcmp (id, GTK_STOCK_MEDIA_FORWARD)) + _id = GTK_STOCK_GO_FORWARD; + else if (!strcmp (id, GTK_STOCK_MEDIA_PREVIOUS) || !strcmp (id, GTK_STOCK_MEDIA_REWIND)) + _id = GTK_STOCK_GO_BACK; + stockMap[cutUnderline (item.label)] = _id; + } // some may not have a stock item because they can't be set on a label // e.g.: gtk-directory, gtk-missing-image, gtk-dnd g_free (id); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r50502 - /trunk/storage/storage/src/modules/Storage.ycp
by aschnell＠svn.opensuse.org 30 Aug '08

30 Aug '08

Author: aschnell Date: Sat Aug 30 11:43:31 2008 New Revision: 50502 URL: http://svn.opensuse.org/viewcvs/yast?rev=50502&view=rev Log: - added TODO note Modified: trunk/storage/storage/src/modules/Storage.ycp Modified: trunk/storage/storage/src/modules/Storage.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/storage/storage/src/modules/Stor… ============================================================================== --- trunk/storage/storage/src/modules/Storage.ycp (original) +++ trunk/storage/storage/src/modules/Storage.ycp Sat Aug 30 11:43:31 2008 @@ -4159,7 +4159,7 @@ - +// TODO: remove /* Contain the current key ( subdevkey ) for all wizards */ string WizardKey = ""; -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r50501 - /trunk/gtk/ChangeLog
by rpmcruz＠svn.opensuse.org 30 Aug '08

30 Aug '08

Author: rpmcruz Date: Sat Aug 30 02:51:31 2008 New Revision: 50501 URL: http://svn.opensuse.org/viewcvs/yast?rev=50501&view=rev Log: Not related with the mentioned bug afterall. Modified: trunk/gtk/ChangeLog Modified: trunk/gtk/ChangeLog URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/ChangeLog?rev=50501&r1=50500… ============================================================================== --- trunk/gtk/ChangeLog (original) +++ trunk/gtk/ChangeLog Sat Aug 30 02:51:31 2008 @@ -13,8 +13,7 @@ * src/YGPackageSelector.cc: bug fix 418896: when filtering a repository, the most appropriate version wasn't the one selected. - * src/YGUtils.cc: fixed truncate() to cope with utf-8 characters. (maybe - related to bug 418781) + * src/YGUtils.cc: fixed truncate() to cope with utf-8 characters. * src/test.cc: added tests for the purpose. * src/YGUtils.cc: replaced the old button icon approach based on English -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r50500 - in /trunk/gtk: ChangeLog src/YGPackageSelector.cc
by rpmcruz＠svn.opensuse.org 30 Aug '08

30 Aug '08

Author: rpmcruz Date: Sat Aug 30 02:51:02 2008 New Revision: 50500 URL: http://svn.opensuse.org/viewcvs/yast?rev=50500&view=rev Log: * src/YGPackageSelector.cc: bug fix 418781: it can happen that zypp doesn't have information on a package repository and we end up with a null pointer. This happens for the (new?) add-on tool, for the added repository. Modified: trunk/gtk/ChangeLog trunk/gtk/src/YGPackageSelector.cc Modified: trunk/gtk/ChangeLog URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/ChangeLog?rev=50500&r1=50499… ============================================================================== --- trunk/gtk/ChangeLog (original) +++ trunk/gtk/ChangeLog Sat Aug 30 02:51:02 2008 @@ -20,6 +20,10 @@ * src/YGUtils.cc: replaced the old button icon approach based on English strings by one that tests button labels to stock ones. + * src/YGPackageSelector.cc: bug fix 418781: it can happen that zypp doesn't + have information on a package repository and we end up with a null pointer. + This happens for the (new?) add-on tool, for the added repository. + 2008-08-22 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> * src/yzypptags.cc: bug request 418676: use title capitalization for Modified: trunk/gtk/src/YGPackageSelector.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGPackageSelector.cc?rev… ============================================================================== --- trunk/gtk/src/YGPackageSelector.cc (original) +++ trunk/gtk/src/YGPackageSelector.cc Sat Aug 30 02:51:02 2008 @@ -1717,8 +1717,13 @@ for (int i = 0; single_package->getAvailableVersion (i); i++) { const Ypp::Package::Version *version = single_package->getAvailableVersion (i); string text = version->number + " <small>(" + version->arch + ")</small>\n"; - string repo = YGUtils::truncate (version->repo->name, - MAX (20, version->number.length() + version->arch.length() + 4), 0); + string repo; + if (version->repo) + repo = YGUtils::truncate (version->repo->name, + MAX (20, version->number.length() + version->arch.length() + 4), 0); + else + yuiError() << "Repository of package '" << single_package->name() + << "' unknown\n"; text += "<small>" + repo + "</small>"; GtkTreeIter iter; gtk_list_store_append (GTK_LIST_STORE (model), &iter); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r50499 - /tags/stable-2_17_1/x11/
by jdsn＠svn.opensuse.org 29 Aug '08

29 Aug '08

Author: jdsn Date: Sat Aug 30 01:01:11 2008 New Revision: 50499 URL: http://svn.opensuse.org/viewcvs/yast?rev=50499&view=rev Log: Created tag stable-2_17_1 for x11 Added: tags/stable-2_17_1/x11/ - copied from r50498, trunk/x11/ -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r50498 - in /trunk/x11: VERSION package/yast2-x11.changes src/XLib.pm src/XMessages.ycp src/autoyast/dialog/autoDialog.ycp src/dialog/x11Dialog.ycp src/proposal/x11_proposal.ycp
by jdsn＠svn.opensuse.org 29 Aug '08

29 Aug '08

Author: jdsn Date: Sat Aug 30 01:00:48 2008 New Revision: 50498 URL: http://svn.opensuse.org/viewcvs/yast?rev=50498&view=rev Log: - using exact display parameters during installation (bnc#399411) and (bnc#331664) - removed 32bit color depth from autoyast (bnc#405797) - tablet setup during installation (fate#302888) - make sure traversal has numeric value (bnc#388259) - cleanup in XMessages strings Modified: trunk/x11/VERSION trunk/x11/package/yast2-x11.changes trunk/x11/src/XLib.pm trunk/x11/src/XMessages.ycp trunk/x11/src/autoyast/dialog/autoDialog.ycp trunk/x11/src/dialog/x11Dialog.ycp trunk/x11/src/proposal/x11_proposal.ycp Modified: trunk/x11/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/VERSION?rev=50498&r1=50497&r… ============================================================================== --- trunk/x11/VERSION (original) +++ trunk/x11/VERSION Sat Aug 30 01:00:48 2008 @@ -1 +1 @@ -2.17.0 +2.17.1 Modified: trunk/x11/package/yast2-x11.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/package/yast2-x11.changes?re… ============================================================================== --- trunk/x11/package/yast2-x11.changes (original) +++ trunk/x11/package/yast2-x11.changes Sat Aug 30 01:00:48 2008 @@ -1,4 +1,19 @@ ------------------------------------------------------------------- +Fri Aug 29 23:02:24 CEST 2008 - jdsn(a)suse.de + +- tablet configuration support during installation (fate#302888) +- make sure traversal has numeric value (bnc#388259) +- consistent dialogs for proposal changes +- cleanup in XMessages strings +- 2.17.1 + +------------------------------------------------------------------- +Thu Aug 22 19:45:39 CEST 2008 - jdsn(a)suse.de + +- using exact display parameters during installation (bnc#399411) +- removed 32bit color depth from autoyast (bnc#405797) + +------------------------------------------------------------------- Tue Jun 24 19:00:47 CEST 2008 - jdsn(a)suse.de - proper type conversion in autoinstallation (bnc#307967) Modified: trunk/x11/src/XLib.pm URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/src/XLib.pm?rev=50498&r1=504… ============================================================================== --- trunk/x11/src/XLib.pm (original) +++ trunk/x11/src/XLib.pm Sat Aug 30 01:00:48 2008 @@ -35,6 +35,7 @@ my %section; my $config; my %cdb; +my %tabletCDB; #========================================== # GetFbColor @@ -258,52 +259,13 @@ my $traversal = $mDesktop->getDisplayTraversal(); my @ratio = @{$mDesktop->getDisplayRatio()}; if (defined $traversal) { - # if traversal is empty, set a minimal default value (bnc#388259) - if ($traversal eq "") { $traversal = 12.2; } - $traversal = sprintf ("%.1f",$traversal); - if ($traversal < 12.2) { - $traversal = 10.0; - } elsif (($traversal >= 12.2) && ($traversal < 13.3)) { - $traversal = 12.2; - } elsif (($traversal >= 13.3) && ($traversal < 14.1)) { - $traversal = 13.3; - } elsif (($traversal >= 14.1) && ($traversal < 14.5)) { - $traversal = 14.1; - } elsif (($traversal >= 14.5) && ($traversal < 15.4)) { - $traversal = 15; - } elsif (($traversal >= 15.4) && ($traversal < 16.5)) { - $traversal = 15.4; - } elsif (($traversal >= 16.5) && ($traversal < 18.0)) { - $traversal = 17; - } elsif (($traversal >= 18.0) && ($traversal < 18.3)) { - $traversal = 18; - } elsif (($traversal >= 18.3) && ($traversal < 18.5)) { - $traversal = 18.1; - } elsif (($traversal >= 18.5) && ($traversal < 19.5)) { - $traversal = 19; - } elsif (($traversal >= 19.5) && ($traversal < 20.5)) { - $traversal = 20; - } elsif (($traversal >= 20.5) && ($traversal < 21.3)) { - $traversal = 21.1; - } elsif (($traversal >= 21.3) && ($traversal < 21.5)) { - $traversal = 21.3; - } elsif (($traversal >= 21.5) && ($traversal < 22.2)) { - $traversal = 22.2; - } elsif (($traversal >= 22.2) && ($traversal < 23.5)) { - $traversal = 23; - } elsif (($traversal >= 23.5) && ($traversal < 24.5)) { - $traversal = 24; - } elsif (($traversal >= 24.5) && ($traversal < 30.5)) { - $traversal = 30; - } elsif (($traversal >= 30.5) && ($traversal < 31.8)) { - $traversal = 31.5; - } elsif (($traversal >= 31.8) && ($traversal < 32.5)) { - $traversal = 32; - } elsif (($traversal >= 32.5) && ($traversal < 40.5)) { - $traversal = 40; - } elsif ($traversal >= 40.5) { - $traversal = 46; - } + # if traversal is empty or not a numeric, set it to undef and do not operate on it (bnc#388259) + if ($traversal eq "" || $traversal !~ /^\d+\.?\d*$/) + { $traversal = "undef"; } + else + # just round the computed (real) traversal to a tenth + { $traversal = sprintf ("%.1f", $traversal); } + @result = ($traversal,@ratio); } return \@result; @@ -898,6 +860,95 @@ } #========================================== +# getTabletCDB +#------------------------------------------ +BEGIN{ $TYPEINFO{getTabletCDB} = ["function",["map","string",["list","string"]]]; } +sub getTabletCDB { + my $class = shift; + my $size = keys %tabletCDB; + if ($size > 0) { + return \%tabletCDB; + } + my $mTablet = new SaX::SaXManipulateTablets ( + $section{Pointers},$section{Layout} + ); + $mTablet->selectPointer (1); + my @vendorList = @{$mTablet->getTabletVendorList()}; + foreach my $vendor (@vendorList) { + my $modelList = $mTablet->getTabletModelList ($vendor); + $tabletCDB{$vendor} = $modelList; + } + return \%tabletCDB; +} +#========================================== +# getTabletVendor +#------------------------------------------ +BEGIN{ $TYPEINFO{getTabletVendor} = ["function", "string"]; } +sub getTabletVendor { + my $class = shift; + my $mTablet = new SaX::SaXManipulateTablets ( + $section{Pointers},$section{Layout} + ); + $mTablet->selectPointer(1); + my $vendor = $mTablet->getVendor(); + if ($vendor =~ /Unknown/i) { + return "undef"; + } + return $vendor; +} +#========================================== +# getTabletModel +#------------------------------------------ +BEGIN{ $TYPEINFO{getTabletModel} = ["function", "string"]; } +sub getTabletModel { + my $class = shift; + my $mTablet = new SaX::SaXManipulateTablets ( + $section{Pointers},$section{Layout} + ); + $mTablet->selectPointer(1); + my $model = $mTablet->getName(); + if ($model =~ /Unknown/i) { + return "undef"; + } + return $model; +} +#========================================== +# getTabletPointer +#------------------------------------------ +BEGIN{ $TYPEINFO{getTabletPointer} = ["function", "string"]; } +sub getTabletPointer { + my $class = shift; + my $mTablet = new SaX::SaXManipulateTablets ( + $section{Pointers},$section{Layout} + ); + # check if one of the first 5 pointer devices is a tablet + my $i; + for ( $i=0 ; $i <= 4 ; $i++ ) + { + $mTablet->selectPointer($i); + if ($mTablet->isTablet() ) + { + return "$i"; + } + } + return "undef"; +} +#========================================== +# setTablet +#------------------------------------------ +BEGIN{ $TYPEINFO{setTablet} = ["function","void",["list","string"]]; } +sub setTablet { + my $vendor = shift; + my $model = shift; + my $mTablet = new SaX::SaXManipulateTablets ( + $section{Pointers},$section{Layout} + ); + $mTablet->selectPointer(1); + $mTablet->setTablet($vendor, $model); +} + + +#========================================== # test code #------------------------------------------ if (0) { Modified: trunk/x11/src/XMessages.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/src/XMessages.ycp?rev=50498&… ============================================================================== --- trunk/x11/src/XMessages.ycp (original) +++ trunk/x11/src/XMessages.ycp Sat Aug 30 01:00:48 2008 @@ -60,17 +60,20 @@ // 65536 colors - upper case string colors65k_uc = _("65536 Colors"); -// 16.7 m colors - lower case -string colors16m_lc = _("16.7 m colors"); -// 16.7 M colors - upper case -string colors16m_uc = _("16.7 M Colors"); -// 16.7 Mio. colors - business language -string colors16m_uc_business = _("16.7 Mio. Colors"); +// 16.7 M colors - lower case +string colors16M_lc = _("16.7 M colors"); +// 16.7 M Colors - upper case +string colors16M_uc = _("16.7 M Colors"); + +// 16.7 Mio. colors - lower case - business language +string colors16Mio_lc_business = _("16.7 Mio. colors"); +// 16.7 Mio. Colors - business language +string colors16Mio_uc_business = _("16.7 Mio. Colors"); global string colors256 = colors256_uc; global string colors32k = colors32k_uc; global string colors65k = colors65k_uc; -global string colors16m = colors16m_uc_business; +global string colors16m = colors16M_uc; //============================================= @@ -78,18 +81,27 @@ global string colors = _("Colors"); // string for a drop-down box -global string color8bit = _("256 [ 8 bit ]"); +global string color8bit = _("256 [8 bit]"); // string for a drop-down box -global string color15bit = _("32768 [ 15 bit ]"); +global string color15bit = _("32768 [15 bit]"); +// string for a drop-down box, PPC meaning: the PowerPC architecture +global string color15bitPPC = _("32768 [15 bit] (PPC)"); // string for a drop-down box -global string color16bit = _("65536 [ 16 bit ]"); -// string for a drop-down box - lower case -string color24bit_lc = _("16.7 m [ 24 bit ]"); -// string for a drop-down box - upper case -string color24bit_uc = _("16.7 M [ 24 bit ]"); +global string color16bit = _("65536 [16 bit]"); +// string for a drop-down box, meaning "16.7 million" (colors) +string color24bitM = _("16.7 M [24 bit]"); // string for a drop-down box - business language -string color24bit_uc_business = _("16.7 Mio. [ 24 bit ]"); -global string color24bit = color24bit_uc_business; +string color24bitMio = _("16.7 Mio. [24 bit]"); +// string for a drop-down box +string color24bitMillion = _("16.7 Million [24 bit]"); +global string color24bit = color24bitM; +// string for a drop-down box, meaning "16.7 million" (colors) +string color32bitM = _("16.7 M [32 bit]"); +// string for a drop-down box - business language +string color32bitMio = _("16.7 Mio. [32 bit]"); +// string for a drop-down box +string color32bitMillion = _("16.7 Million [32 bit]"); +global string color32bit = color32bitM; //============================================= // Xgl strings @@ -123,7 +135,8 @@ //============================================= // colorDepth //--------------------------------------------- -global string colorDepth = _("Colordepth: "); +global string colorDepthOld = _("Colordepth: "); +global string colorDepth = _("Color Depth: "); //============================================= // deactivated @@ -151,6 +164,11 @@ global string selectColorDepth = _("Select color depth:"); //============================================= +// resolution +//--------------------------------------------- +global string resolution = _("Resolution:"); + +//============================================= // selectResolution //--------------------------------------------- global string selectResolution = _("Select resolution:"); @@ -158,7 +176,19 @@ //============================================= // selectMonitor //--------------------------------------------- -global string selectMonitor = _("Select Monitor Vendor and Model:"); +global string selectMonitorOld = _("Select Monitor Vendor and Model:"); +global string selectMonitor = _("Select Monitor Vendor and Model"); + +// selectTablet +global string selectTablet = _("Select Tablet Vendor and Model"); + +// tablet +global string Tablet = _("Tablet:"); + +// vendor - meaning "monitor vendor" (like "hardware vendor") +global string Vendor = _("Vendor:"); +// model - meaning "monitor model" +global string Model = _("Model:"); //============================================= // selectLink @@ -178,9 +208,11 @@ //============================================= // selectDisplaySize //--------------------------------------------- -global string selectDisplayTraversal = _("Traversal") + " [ " + Inches + " ]:"; +global string selectDisplayTraversal = _("Traversal") + " [" + Inches + "]:"; global string selectDisplayRatio = _("Aspect Ratio:"); +global string displaySettings = _("Display Settings"); + //============================================= // proposal menu and title text //--------------------------------------------- @@ -197,6 +229,9 @@ //--------------------------------------------- global string unconfigured = _("Unconfigured"); +// configure - in lower case +global string configure = _("configure"); + //============================================= // warnNoteBook //--------------------------------------------- Modified: trunk/x11/src/autoyast/dialog/autoDialog.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/src/autoyast/dialog/autoDial… ============================================================================== --- trunk/x11/src/autoyast/dialog/autoDialog.ycp (original) +++ trunk/x11/src/autoyast/dialog/autoDialog.ycp Sat Aug 30 01:00:48 2008 @@ -12,6 +12,7 @@ import "AutoinstX11"; import "Wizard"; import "Popup"; + import "XMessages"; /** * Build main Dialog @@ -44,27 +45,17 @@ "2800x2100 (QSXGA+)", "3200x2400 (QUXGA)" ]; + + // made color depth list consistent and removed the 32bit special settings (bnc#405797) list<map> depths_list = [ - $[ - "desc" :_("8 Bit / 256 Colors"), - "depth" : 8 - ], - $[ - "desc" :_("15 Bit / 32 K Colors (PPC)"), - "depth" : 15 - ], - $[ - "desc" :_("16 Bit / 64 K Colors"), - "depth" : 16 - ], - $[ - "desc" :_("24 Bit / 16 M Colors"), - "depth" : 24 - ], - $[ - "desc" :_("32 Bit / 16 M Colors"), - "depth" : 32 - ] + $[ "desc" : XMessages::color8bit, + "depth" : 8 ], + $[ "desc" : XMessages::color15bit, + "depth" : 15 ], + $[ "desc" : XMessages::color16bit, + "depth" : 16 ], + $[ "desc" : XMessages::color24bit, + "depth" : 24 ] ]; @@ -75,7 +66,7 @@ foreach(map d, depths_list , ``{ color_depths=add(color_depths, - `item( `id(d["depth"]:16), d["desc"]:_("16 Bit / 64 K Colors") , + `item( `id(d["depth"]:16), d["desc"]:XMessages::color16bit, ( d["depth"]:16 == x11["color_depth"]:16))); }); list displaymanager = Modified: trunk/x11/src/dialog/x11Dialog.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/src/dialog/x11Dialog.ycp?rev… ============================================================================== --- trunk/x11/src/dialog/x11Dialog.ycp (original) +++ trunk/x11/src/dialog/x11Dialog.ycp Sat Aug 30 01:00:48 2008 @@ -48,188 +48,348 @@ //================================ // setupColors //-------------------------------- -define string setupColors ( string currentRes, string currentText ) ``{ - string space = " "; - string topic = XMessages::selectColorDepth + space; - UI::OpenDialog( - `VBox( - `HBox(`ComboBox(`id(`depth),`opt(`hstretch),topic, - [ - `item(`id(`bit8), "256 [ 8 bit ]" ), - `item(`id(`bit15), "32768 [ 15 bit ]" ), - `item(`id(`bit16), "65536 [ 16 bit ]" ), - `item(`id(`bit24), "16.7 Mio. [ 24 bit ]" ) - ] - )), - `Right (`HBox ( - `PushButton(`id(`ok),Label::OKButton()), - `PushButton(`id(`cancel),Label::CancelButton()) - )) - ) - ); - while (true) { - UI::ChangeWidget(`id(`depth), `Value, currentText); - any id = UI::UserInput(); - if (id == `cancel) { - UI::CloseDialog(); - if (currentText == "256 [ 8 bit ]") { - return "8"; - } - if (currentText == "32768 [ 15 bit ]") { - return "15"; - } - if (currentText == "65536 [ 16 bit ]") { - return "16"; - } - if (currentText == "16.7 Mio. [ 24 bit ]") { - return "24"; - } - return "undef"; - } - any col=UI::QueryWidget(`id(`depth), `Value); - string color = "8"; - if ( col == `bit8 ) color = "8"; - if ( col == `bit15 ) color = "15"; - if ( col == `bit16 ) color = "16"; - if ( col == `bit24 ) color = "24"; - if (! XLib::hasValidColorResolutionSetup (color,currentRes)) { - warnInvalidColorResolution(); - continue; - } - UI::CloseDialog(); - return (string)color; - } +define string setupColors ( string currentRes, string currentDepth ) +{ // rewritten due to consistency with setupDisplaySize + + string topic = XMessages::colorDepth; + UI::OpenDialog( `MarginBox( 0.5, 0.5, + `VBox( + `Heading(XMessages::displaySettings), + `HBox(`ComboBox(`id(`depth),`opt(`hstretch), topic, + [ + `item(`id("8"), XMessages::color8bit ), + `item(`id("15"), XMessages::color15bit ), + `item(`id("16"), XMessages::color16bit ), + `item(`id("24"), XMessages::color24bit ), + ] + )), + `VSpacing(0.5), + `HBox ( + `PushButton(`id(`ok),Label::OKButton()), + `PushButton(`id(`cancel),Label::CancelButton()) + ) + ) + )); + + string color = "8"; + if (contains(["8", "15", "16", "24"], currentDepth)) + { color = currentDepth; } + else + { currentDepth = "undef"; } + UI::ChangeWidget(`id(`depth), `Value, color); + + any ret = nil; + + repeat + { + ret = UI::UserInput(); + + color = (string) UI::QueryWidget(`id(`depth), `Value); + + if (! XLib::hasValidColorResolutionSetup (color,currentRes)) + { + warnInvalidColorResolution(); + ret = nil; + } + + } until ( ret == `ok || ret == `cancel); + + UI::CloseDialog(); + if (ret == `cancel) + { + UI::CloseDialog(); + return currentDepth; + } + + return color; + }; //================================ // setupResolutions //-------------------------------- -define string setupResolutions ( - list reslist, string currentText, string currentColor -) ``{ - string space = " "; - string topic = XMessages::selectResolution + space; - UI::OpenDialog( - `VBox( - `HBox(`Left ( `ComboBox(`id(`resitem),topic, reslist))), - `Right (`HBox ( - `PushButton(`id(`ok),Label::OKButton()), - `PushButton(`id(`cancel),Label::CancelButton()) - )) - ) - ); - while (true) { - UI::ChangeWidget(`id(`resitem), `Value, currentText); - any id = UI::UserInput(); - if (id == `cancel) { - UI::CloseDialog(); - return currentText; - } - any res=UI::QueryWidget(`id(`resitem), `Value); - if (! XLib::hasValidColorResolutionSetup (currentColor,(string)res)) { - warnInvalidColorResolution(); - continue; - } - UI::CloseDialog(); - return (string)res; - } +define string setupResolutions (list reslist, string currentText, string currentColor) +{ // rewritten to be consistent with + + string topic = XMessages::resolution; + UI::OpenDialog( `MarginBox( 0.5, 0.5, + `VBox( + `Heading(XMessages::displaySettings), + `VSpacing(0.5), + `ComboBox(`id(`resitem),topic, reslist), + `VSpacing(0.5), + `HBox ( + `PushButton(`id(`ok),Label::OKButton()), + `PushButton(`id(`cancel),Label::CancelButton()) + ) + ) + )); + + UI::ChangeWidget(`id(`resitem), `Value, currentText); + any ret = nil; + string res = ""; + + repeat + { + ret = UI::UserInput(); + + res = (string) UI::QueryWidget(`id(`resitem), `Value); + if (! XLib::hasValidColorResolutionSetup(currentColor, res)) + { + warnInvalidColorResolution(); + ret = nil; + } + } until (ret == `ok || ret == `cancel); + + UI::CloseDialog(); + if (ret == `cancel) { return currentText; } + return res; + }; //================================ // setupDisplaySize //-------------------------------- -define list<string> setupDisplaySize ( - string currentTraversal, string currentRatios -) ``{ - //string space = " "; - string topic1 = XMessages::selectDisplayTraversal; - string topic2 = XMessages::selectDisplayRatio; - list<string> traverlist = ["21","20","19","18.1","17","15.4","15","14.1","13.3","12.2"]; - list<string> ratioslist = ["4/3","5/4","16/10"]; - UI::OpenDialog( - `VBox( - `HBox( - `Left ( `ComboBox(`id(`traveritem),topic1, traverlist)), - `Right ( `ComboBox(`id(`ratiositem),topic2, ratioslist)) - ), - `Right (`HBox ( - `PushButton(`id(`ok),Label::OKButton()), - `PushButton(`id(`cancel),Label::CancelButton()) - )) - ) - ); - UI::ChangeWidget(`id(`traveritem), `Value, currentTraversal); - UI::ChangeWidget(`id(`ratiositem), `Value, currentRatios); - any id = UI::UserInput(); - list<string> result = []; - if (id == `cancel) { - UI::CloseDialog(); - result = add (result,currentTraversal); - result = add (result,currentRatios); - return result; - } - any traversal=UI::QueryWidget(`id(`traveritem), `Value); - any ratios=UI::QueryWidget(`id(`ratiositem), `Value); - result = add (result,(string)traversal); - result = add (result,(string)ratios); - UI::CloseDialog(); - return result; +define list<string> setupDisplaySize (string currentTraversal, string currentRatio) +{ + // function rewritten because of (bnc#331664) and (bnc#399411) + + string topic1 = XMessages::selectDisplayTraversal; + string topic2 = XMessages::selectDisplayRatio; + // offer some common display settings + list<string> traversallist = [ "32.0", "31.5", "30.0", "26.0", "24.0", "23.0", + "22.2", "21.3", "21.1", "20.0", "19.0", "18.1", + "17.0", "15.4", "15.0", "14.1", "13.3", "12.2", "10.0" ]; + list<string> ratiolist = [ "4/3", "5/4", "16/9", "16/10" ]; + + string traversal = currentTraversal; + string ratio = currentRatio; + string traversalCheck = ""; + string ratioCheck = ""; + + // merge calculated values into the predefined lists + traversallist = (list<string>) sort(union(traversallist, [traversal])); + ratiolist = (list<string>) sort(union(ratiolist, [ratio])); + + UI::OpenDialog(`MarginBox(0.5, 0.5, + `VBox( + `Heading(_("Display Settings")), + `VSpacing(0.5), + `HBox( + `HSpacing(`opt(`hstretch), 0.5), + `VBox( + `Left( `ComboBox(`id(`traversal), `opt(`notify, `editable), topic1, traversallist)), + `Left( `ComboBox(`id(`ratio), `opt(`notify, `editable), topic2, ratiolist)) + ), + `HSpacing(`opt(`hstretch), 0.5) + ), + `VSpacing(0.5), + `HBox ( + `PushButton(`id(`ok),Label::OKButton()), + `PushButton(`id(`cancel),Label::CancelButton()) + ) + ) + )); + + UI::ChangeWidget(`id(`traversal), `ValidChars, "1234567890."); + UI::ChangeWidget(`id(`ratio), `ValidChars, "1234567890/"); + + UI::ChangeWidget(`id(`traversal), `Value, traversal); + UI::ChangeWidget(`id(`ratio), `Value, ratio); + + + any ret = nil; + repeat + { + // wait for event from user + ret = UI::UserInput(); + + traversal = (string) UI::QueryWidget(`id(`traversal), `Value); + ratio = (string) UI::QueryWidget(`id(`ratio), `Value); + + traversalCheck = regexpsub(traversal, "([1-9]+[0-9]*)(\.[0-9]?)?", "\\1\\2"); + if (traversalCheck == nil) { traversal = ""; traversalCheck = ""; } + if (traversalCheck != traversal) + { + traversal = traversalCheck; + UI::ChangeWidget(`id(`traversal), `Value, traversal); + } + + // here for ratio just sanity check + ratioCheck = regexpsub(ratio, "([1-9]+[0-9]*)(\/([1-9]+[0-9]*)?)?", "\\1\\2"); + if (ratioCheck == nil) { ratio = ""; ratioCheck = ""; } + if (ratioCheck != ratio) + { + ratio = ratioCheck; + UI::ChangeWidget(`id(`ratio), `Value, ratio); + } + + if (ret == `ok) + { + string ratioOrig = ratio; + ratio = regexpsub(ratio, "([1-9]+[0-9]*)\/([1-9]+[0-9]*)", "\\1\/\\2"); + if (ratio != ratioOrig || ratioOrig == "" || ratio == nil) + { + Popup::Message(_("The value for the display ratio is invalid, please correct.")); + ret = nil; + } + + if (traversal == "" || traversal == nil) + { + Popup::Message(_("The value for the display traversal is invalid, please correct.")); + ret = nil; + } + } + + } until (ret == `ok || ret == `cancel); + + + list<string> result = [traversal, ratio]; + if (ret == `cancel) + { + result = [currentTraversal, currentRatio]; + } + + UI::CloseDialog(); + return result; }; //================================ // setupMonitor //-------------------------------- -define list setupMonitor ( - map<string,list> data, string currentVendor, string currentModel -) ``{ - any id = nil; - list<string> result = []; - list<string> vendorList = []; - foreach (string vendor, list models, data, ``{ - vendorList = vendorList + vendor; - }); - UI::OpenDialog( - `VBox( - `HSpacing(70), - `Left (`Label(XMessages::selectMonitor)), - `HBox ( - `VSpacing(20), - `SelectionBox(`id(`vendor),`opt(`notify,`immediate),"",vendorList), - `SelectionBox(`id(`model) ,`opt(`notify,`immediate),"", []) - ), - `Right( `HBox ( - `PushButton(`id(`ok),Label::OKButton()), - `PushButton(`id(`cancel),Label::CancelButton()) - )) - ) - ); - list modelList = sort (data[currentVendor]:[]); - UI::ChangeWidget(`id(`vendor), `CurrentItem, currentVendor); - UI::ChangeWidget(`id(`model) , `Items, modelList); - UI::ChangeWidget(`id(`model) , `CurrentItem, currentModel); - any vendor = (string)currentVendor; - any model = (string)currentModel; - repeat { - id = UI::UserInput(); - if (id == `vendor) { - vendor = UI::QueryWidget(`id(`vendor), `CurrentItem); - list modelList = sort (data[(string)vendor]:[]); - UI::ChangeWidget( `id( `model ), `Items, [] ); - UI::ChangeWidget( `id( `model ), `Items, modelList ); - } - if (id == `model) { - model = UI::QueryWidget(`id(`model), `CurrentItem); - } - if (id == `cancel) { - result[0] = (string)currentVendor; - result[1] = (string)currentModel; - UI::CloseDialog(); - return result; - } - } until ( id == `ok ); - UI::CloseDialog(); - result[0] = (string)vendor; - result[1] = (string)model; - return result; +define list setupMonitor ( map<string,list> data, string currentVendor, string currentModel ) +{ + any ret = nil; + list<string> result = []; + list<string> vendorList = []; + list modelList = []; + + vendorList = maplist( string Dvendor, list Dmodel, data, { return Dvendor; }); + + UI::OpenDialog(`MarginBox( 0.5, 0.5, + `VBox( + `Left (`Label(XMessages::selectMonitor)), + `MinSize( 75, 15, `HBox ( + `SelectionBox(`id(`vendor),`opt(`notify,`immediate), XMessages::Vendor,vendorList), + `SelectionBox(`id(`model) ,`opt(`notify,`immediate), XMessages::Model, []) + )), + `VSpacing(0.5), + `HBox ( + `PushButton(`id(`ok),Label::OKButton()), + `PushButton(`id(`cancel),Label::CancelButton()) + ) + ) + )); + + string vendor = currentVendor; + string model = currentModel; + + modelList = sort (data[vendor]:[]); + UI::ChangeWidget(`id(`vendor), `CurrentItem, nil); + UI::ChangeWidget(`id(`model), `CurrentItem, nil); + + if ( modelList != [] && modelList != nil) + { + UI::ChangeWidget(`id(`vendor), `CurrentItem, vendor); + UI::ChangeWidget(`id(`model) , `Items, modelList); + UI::ChangeWidget(`id(`model), `CurrentItem, nil); + + if (contains(modelList, model)) + UI::ChangeWidget(`id(`model) , `CurrentItem, model); + } + + repeat { + ret = UI::UserInput(); + if (ret == `vendor) { + vendor = (string) UI::QueryWidget(`id(`vendor), `CurrentItem); + list modelList = sort (data[vendor]:[]); + UI::ChangeWidget(`id(`model ), `Items, modelList ); + UI::ChangeWidget(`id(`model), `CurrentItem, nil); + } + if (ret == `model) { + model = (string) UI::QueryWidget(`id(`model), `CurrentItem); + } + if (ret == `cancel) { + UI::CloseDialog(); + return [currentVendor, currentModel]; + } + } until ( ret == `ok ); + + UI::CloseDialog(); + return [vendor, model]; + }; + + +//================================ +// setupTablet (fate#302888) +//-------------------------------- +define list<string> setupTablet ( map<string,list> data, string currentVendor, string currentModel ) +{ + any ret = nil; + list<string> result = []; + list<string> vendorList = []; + list modelList = []; + + vendorList = maplist( string Dvendor, list Dmodel, data, { return Dvendor; }); + + UI::OpenDialog(`MarginBox( 0.5, 0.5, + `VBox( + `Left (`Label(XMessages::selectTablet)), + `MinSize( 75, 15, `HBox ( + `SelectionBox(`id(`vendor),`opt(`notify,`immediate), XMessages::Vendor, vendorList), + `SelectionBox(`id(`model) ,`opt(`notify,`immediate), XMessages::Model, []) + )), + `VSpacing(0.5), + `HBox ( + `PushButton(`id(`ok),Label::OKButton()), + `PushButton(`id(`cancel),Label::CancelButton()) + ) + ) + )); + + string vendor = currentVendor; + string model = currentModel; + + modelList = sort (data[vendor]:[]); + UI::ChangeWidget(`id(`vendor), `CurrentItem, nil); + UI::ChangeWidget(`id(`model), `CurrentItem, nil); + + if ( modelList != [] && modelList != nil) + { + UI::ChangeWidget(`id(`vendor), `CurrentItem, vendor); + UI::ChangeWidget(`id(`model) , `Items, modelList); + UI::ChangeWidget(`id(`model), `CurrentItem, nil); + + if (contains(modelList, model)) + UI::ChangeWidget(`id(`model) , `CurrentItem, model); + } + + repeat + { + ret = UI::UserInput(); + if (ret == `vendor) + { + vendor = (string) UI::QueryWidget(`id(`vendor), `CurrentItem); + modelList = sort (data[vendor]:[]); + UI::ChangeWidget(`id(`model), `Items, modelList ); + UI::ChangeWidget(`id(`model), `CurrentItem, nil); + } + if (ret == `model) + { + model = (string) UI::QueryWidget(`id(`model), `CurrentItem); + } + if (ret == `cancel) + { + UI::CloseDialog(); + return [currentVendor, currentModel]; + } + } until ( ret == `ok ); + + UI::CloseDialog(); + return [vendor, model]; +} + + + } Modified: trunk/x11/src/proposal/x11_proposal.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/src/proposal/x11_proposal.yc… ============================================================================== --- trunk/x11/src/proposal/x11_proposal.ycp (original) +++ trunk/x11/src/proposal/x11_proposal.ycp Sat Aug 30 01:00:48 2008 @@ -98,7 +98,7 @@ //-------------------------------------- if (! (boolean)XLib::isInitialized()) { y2milestone ("Loading library cache..."); - Popup::TimedMessage(XMessage::probeWarning, 15); + Popup::TimedMessage(XMessages::probeWarning, 15); XLib::loadApplication(); } //====================================== @@ -117,6 +117,9 @@ boolean extVGAactive= (boolean) XLib::isExternalVGAactive(); list monitorSize = (list) XLib::getDisplaySize(); string monitorInch = (string) monitorSize[0]:"undef"; + //string tabletPtr = (string) XLib::getTabletPointer(); + string tabletName = (string) XLib::getTabletModel(); + string tabletVendor= (string) XLib::getTabletVendor(); //====================================== // Prepare proposal @@ -217,6 +220,20 @@ //warnNoteBookPopup(); } } + //====================================== + // Tablet settings (fate#302888) + //-------------------------------------- + string tabletStatus = XMessages::unconfigured; + if ( tabletVendor != "undef" && + tabletName != "undef" ) + { + tabletStatus = sformat("%1 - %2", tabletVendor, tabletName); + } + proposal = proposal + + LI + + XMessages::Tablet + " " + tabletStatus + + " (<a href=\"sax-tablet\">" + XMessages::configure + "</a>)" + + IL; //====================================== // Test your configuration now //-------------------------------------- @@ -241,7 +258,8 @@ "sax-colors", "sax-3d", "sax-ext-vga", - "sax-test" + "sax-test", + "sax-tablet" ] ]; return ret; @@ -340,6 +358,15 @@ if (chosenID == "x11_conf") { infoPopup(); } + if (chosenID == "sax-tablet" ) { + y2milestone("X11: setup tablet..."); + string vendor = XLib::getTabletVendor(); + string model = XLib::getTabletModel(); + map<string,list> tcdb = XLib::getTabletCDB(); + list selectedTablet = setupTablet(tcdb, vendor, model); + y2milestone("X11: setup tablet <%1>", selectedTablet); + XLib::setTablet( (list<string>)selectedTablet ); + } } //=================================================== // Handle proposal description... -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0