Skip to content

Commit

Permalink
SmartKeys with KB disabled
Browse files Browse the repository at this point in the history
- Showing SmartKeys with the hardware keyboard has been working in a
non-consistent way since the last iOS. We are disabling it as we kinda have to
decide between having the keyboard bar properly hidden completely or this. And
the keys were not useful on iPad anyway. We will figure out a better way for iOS18.
  • Loading branch information
Carlos Cabanero committed Apr 8, 2024
1 parent ac1f002 commit fb6a4bb
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 4 additions & 2 deletions Blink/KBTracker.swift
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,8 @@ class KBObserver: NSObject, UIInteraction {
}

class KBTracker: NSObject {
private(set) var hideSmartKeysWithHKB = !BKUserConfigurationManager.userSettingsValue(forKey: BKUserConfigShowSmartKeysWithXKeyBoard)
private(set) var hideSmartKeysWithHKB = true
//private(set) var hideSmartKeysWithHKB = !BKUserConfigurationManager.userSettingsValue(forKey: BKUserConfigShowSmartKeysWithXKeyBoard)

@objc static let shared = KBTracker()

Expand Down Expand Up @@ -229,7 +230,8 @@ class KBTracker: NSObject {
}

@objc private func _updateSettings() {
hideSmartKeysWithHKB = !BKUserConfigurationManager.userSettingsValue(forKey: BKUserConfigShowSmartKeysWithXKeyBoard)
hideSmartKeysWithHKB = true
//hideSmartKeysWithHKB = !BKUserConfigurationManager.userSettingsValue(forKey: BKUserConfigShowSmartKeysWithXKeyBoard)

input?.sync(traits: kbTraits, device: kbDevice, hideSmartKeysWithHKB: hideSmartKeysWithHKB)
}
Expand Down
2 changes: 1 addition & 1 deletion Settings/Base.lproj/Settings.storyboard
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
<sections>
<tableViewSection footerTitle="Note: on iPads, Smart Keys bar is visible only when Settings.app → General → Keyboard → Shortcuts toggle is on." id="4LL-aT-m70">
<cells>
<tableViewCell clipsSubviews="YES" contentMode="scaleToFill" preservesSuperviewLayoutMargins="YES" selectionStyle="none" indentationWidth="10" rowHeight="44" id="Pas-GB-VuG">
<tableViewCell hidden="YES" clipsSubviews="YES" contentMode="scaleToFill" preservesSuperviewLayoutMargins="YES" selectionStyle="none" indentationWidth="10" rowHeight="44" id="Pas-GB-VuG">
<rect key="frame" x="0.0" y="18" width="768" height="44"/>
<autoresizingMask key="autoresizingMask"/>
<tableViewCellContentView key="contentView" opaque="NO" clipsSubviews="YES" multipleTouchEnabled="YES" contentMode="center" preservesSuperviewLayoutMargins="YES" tableViewCell="Pas-GB-VuG" id="MJS-q5-cvs">
Expand Down

0 comments on commit fb6a4bb

Please sign in to comment.