Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • KWin KWin
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 44
    • Issues 44
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 115
    • Merge requests 115
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Plasma
  • KWinKWin
  • Merge requests
  • !1954

kcm/kwinrules: Improve button behavior when adding properties

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Ismael Asensio requested to merge work/iasensio/ruleseditor-properties-button into master Jan 28, 2022
  • Overview 4
  • Commits 1
  • Pipelines 2
  • Changes 1
  • Make the button on each delegate visible not only on hover to be more consistent with current status everywhere else.

  • Change the icon when the property is already added and this will just apply the suggestion. Using edit-redo as a lack of a better edit-apply or edit-replace icon.

  • Keep properties sheet open when clicking on the button to add a new property. This allows to add several properties in a row. Clicking on the full row will keep the previous behavior for the simple case, closing the sheet so the new property can be edited right away

Previously it worked the same for both items, so the button was merely informative and the sheet would remain open only after detecting some window properties, which was a very hidden and confusing pattern.

Not CCBUGing on purpose, since while the idea is good, it comes from a rant-formed catch-it-all bug report.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: work/iasensio/ruleseditor-properties-button