FreshRSS
/
NetNewsWire
mirror of https://github.com/Ranchero-Software/NetNewsWire.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2516 from Wevah/main 

Crasher fixes (main branch)
This commit is contained in:
Maurice Parker 2020-10-20 14:14:56 -05:00 committed by GitHub
parent 006d02a824 643b9f4659
commit a38e7f0182
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Mac/Preferences/General/GeneralPrefencesViewController.swift
View File

@ -202,10 +202,12 @@ private extension GeneralPreferencesViewController {
menu.addItem(NSMenuItem.separator())
for browser in allBrowsers {
let item = NSMenuItem(title: browser.name!, action: nil, keyEquivalent: "")
guard let name = browser.name else { continue }
let item = NSMenuItem(title: name, action: nil, keyEquivalent: "")
item.representedObject = browser.bundleIdentifier
let icon = browser.icon!
let icon = browser.icon ?? NSWorkspace.shared.icon(forFileType: kUTTypeApplicationBundle as String)
icon.size = NSSize(width: 16.0, height: 16.0)
item.image = browser.icon
menu.addItem(item)

2
Shared/Extensions/NSAttributedString+NetNewsWire.swift
View File

@ -191,6 +191,8 @@ extension NSAttributedString {
result.addAttribute(.font, value: baseFont, range: NSRange(location: 0, length: result.length))
for (range, styles) in attributeRanges {
if range.location >= result.length { continue }
let currentFont = result.attribute(.font, at: range.location, effectiveRange: nil) as! Font
let currentDescriptor = currentFont.fontDescriptor
var descriptor = currentDescriptor.copy() as! FontDescriptor