Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify the button sets #1127

Closed
wants to merge 16 commits into from
Closed

Conversation

dgdavid
Copy link
Member

@dgdavid dgdavid commented Sep 17, 2020

Problem

Short description of the original problem.

  • Bugzilla link
  • openQA link
  • Links to other related pull requests

Solution

Short description of the fix.

Testing

  • Added a new unit test
  • Tested manually

Screenshots

If the fix affects the UI attach some screenshots here.

Instead of being a DeviceMenuButton grouping multiple actions, from now
on it is just a DeviceButton to trigger the modify action for the
selected device.
To simplify the DeviceButtonsSet using a simple DeviceButton for directly
add a partition instead of a DeviceMenuButton grouping several
partitions actions.
To simplify the DeviceButtonsSet using a simple DeviceButton for
directly modify the device instead of a DeviceMenuButton grouping
several related actions.

Related to improvements discussed in
https://lists.opensuse.org/yast-devel/2020-09/msg00004.html
To simplify the DeviceButtonsSet using a simple DeviceButton for
directly modify the device instead of a DeviceMenuButton grouping
several related actions.

Related to improvements discussed in
https://lists.opensuse.org/yast-devel/2020-09/msg00004.html
To simplify the DeviceButtonsSet when an LVM volume group is selected,
according to the improvements discussed in
https://lists.opensuse.org/yast-devel/2020-09/msg00004.html
To simplify the DeviceButtonsSet using a simple DeviceButton for
directly adding an LVM LV instead of a DeviceMenuButton grouping
several LVM VG related actions.

Related to improvements discussed in
https://lists.opensuse.org/yast-devel/2020-09/msg00004.html
Instead of being a DeviceMenuButton grouping multiple actions, it is now
a DeviceButton to trigger the modify action for the selected LVM logical
volume.

Related to improvements discussed in
https://lists.opensuse.org/yast-devel/2020-09/msg00004.html
Instead of being a DeviceMenuButton grouping multiple actions, it is now
a DeviceButton to trigger the modify action for the selected Btrfs
device.

Related to improvements discussed in
https://lists.opensuse.org/yast-devel/2020-09/msg00004.html
Instead of being a DeviceMenuButton grouping multiple actions, it is now
a DeviceButton to trigger the modify action for selected partition.

Related to improvements discussed in
https://lists.opensuse.org/yast-devel/2020-09/msg00004.html
Replacing the partition menu-button by the button to create a new
partition table, according to improvements discussed in
https://lists.opensuse.org/yast-devel/2020-09/msg00004.html
Because they are no longer needed after latest UI changes to start using
a menu bar.

To know more, visit

  * #1125, and
  * https://lists.opensuse.org/yast-devel/2020-09/msg00004.html
Not calling neither, MainMenuBar#select_row nor MainMenuBar#select_page,
when the main menu bar is not ready yet.
@dgdavid dgdavid closed this Sep 21, 2020
@dgdavid dgdavid deleted the simplify-button-sets branch September 21, 2020 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant