Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • manjaro-settings-manager manjaro-settings-manager
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 10
    • Issues 10
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ApplicationsApplications
  • manjaro-settings-managermanjaro-settings-manager
  • Merge requests
  • !117

TimeDate: Fix "Apply" button enabling for checkbox options (fixes #111)

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Philip Müller requested to merge jhrdina:fix-ntp-apply-button into master Oct 16, 2017
  • Overview 5
  • Commits 1
  • Pipelines 0
  • Changes 2

Created by: jhrdina

Fixes #111 (closed).

setApplyEnabled( this, true ) call was missing inside isNtpEnabledCheckBox's toggled signal handler.

Signals are now blocked during initial values setup to prevent firing of unwanted QCheckBox::toggled signal and therefore making Apply button visible even if user doesn't make any changes.

Issue was fixed even for isRtcLocalCheckBox.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: jhrdina:fix-ntp-apply-button