Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • libhandy libhandy
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 131
    • Issues 131
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 19
    • Merge requests 19
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

  • Librem5
  • libhandylibhandy
  • Merge requests
  • !204

combo-row: Rework selected-index property setting and notification

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Benjamin Berg requested to merge benzea/libhandy:fix-selection-index into master Jan 17, 2019
  • Overview 6
  • Commits 1
  • Pipelines 3
  • Changes 1

The notify::selected-index signal was not selected in most cases. Rework the selection handling to ensure that it is always emited when it changes or if the module is replaced.

Also fixed are a few checks on whether the selection index is valid.

Edited Jan 17, 2019 by Benjamin Berg
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: fix-selection-index