forked from openlp/openlp
In order to add an image to an existing group when no group was preselected, the user must currently choose the existing group name from the comboxbox and also select the Existing Group radio button. It should be assumed that by selecting a group name from the combobox, the user intendeds to add the image to an existing group, and the accompanying radio button should automatically be selected. This reduces the number of required clicks, and the likelihood of not actually adding the image to the correct group. Likewise, if a user enters text into the New Group field, the dialog should assume that the user's intent is to create a new group and auto select the appropriate radio button. Also removes some choosegroupdialog specific component logic from mediaitem, since it's now covered by the choosegroupdialog implementation. Better encapsulation, and improves testability. (Testing that the existing group radio button was selected when choosedialogform was initialized with a preselected group requires much more effort when the radio button selection logic spanned two components.) Adds simple test cases for the scenarios described above.
22 lines
1.5 KiB
Python
22 lines
1.5 KiB
Python
# -*- coding: utf-8 -*-
|
|
# vim: autoindent shiftwidth=4 expandtab textwidth=120 tabstop=4 softtabstop=4
|
|
|
|
###############################################################################
|
|
# OpenLP - Open Source Lyrics Projection #
|
|
# --------------------------------------------------------------------------- #
|
|
# Copyright (c) 2008-2018 OpenLP Developers #
|
|
# --------------------------------------------------------------------------- #
|
|
# This program is free software; you can redistribute it and/or modify it #
|
|
# under the terms of the GNU General Public License as published by the Free #
|
|
# Software Foundation; version 2 of the License. #
|
|
# #
|
|
# This program is distributed in the hope that it will be useful, but WITHOUT #
|
|
# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or #
|
|
# FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for #
|
|
# more details. #
|
|
# #
|
|
# You should have received a copy of the GNU General Public License along #
|
|
# with this program; if not, write to the Free Software Foundation, Inc., 59 #
|
|
# Temple Place, Suite 330, Boston, MA 02111-1307 USA #
|
|
###############################################################################
|