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

Redesign toolbar #2883

Merged
merged 5 commits into from
Nov 14, 2023
Merged

Redesign toolbar #2883

merged 5 commits into from
Nov 14, 2023

Conversation

iiLubos
Copy link
Contributor

@iiLubos iiLubos commented Oct 30, 2023

New toolbar behavior:

  • at least 4 buttons will always be visible
  • all the others will go automatically into the menu
  • the long button has defined minimum and maximum size
Toolbar_desktop.mov
Toolbar_mobile.mp4
  MMToolbar {
    model: ObjectModel {
      MMToolbarButton { text: qsTr("Delete"); iconSource: Style.deleteIcon }
      MMToolbarButton { text: qsTr("Edit Geometry"); iconSource: Style.editIcon }
      MMToolbarButton { text: qsTr("Save"); iconSource: Style.doneIcon; type: MMToolbarButton.Button.Save }
    }
  }

  MMToolbar {
    model: ObjectModel {
      MMToolbarLongButton { text: "Long button"; iconSource: Style.deleteIcon }
    }
  }

The design of the Drawer is defined in MMMenuDrawer.qml
and its menu buttons in MMToolbarMenuButton.qml

app/qmlV2/component/MMToolbar.qml Show resolved Hide resolved
app/qmlV2/component/MMToolbar.qml Show resolved Hide resolved
app/qmlV2/component/MMToolbar.qml Outdated Show resolved Hide resolved
app/qmlV2/component/MMToolbar.qml Outdated Show resolved Hide resolved
app/qmlV2/component/MMToolbar.qml Outdated Show resolved Hide resolved
iiLubos and others added 3 commits November 14, 2023 12:23
Co-authored-by: Tomas Mizera <[email protected]>
Co-authored-by: Tomas Mizera <[email protected]>
Co-authored-by: Tomas Mizera <[email protected]>
@tomasMizera tomasMizera merged commit 346b644 into master Nov 14, 2023
9 checks passed
@tomasMizera tomasMizera deleted the master_redesign-toolbar branch November 14, 2023 11:31
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.

2 participants