fedfb24220
Merge pull request #1506 from Wevah/ios-blockquote-tweaks
...
iPhone 8 Plus viewport is actually 414px wide
2019-12-31 18:37:58 -08:00
53b2ebab78
iPhone 8 Plus viewport is actually 414px wide
2019-12-31 20:30:37 -06:00
c2a7f8501e
Merge pull request #1505 from Wevah/ios-blockquote-tweaks
...
Don't shrink margins for wider viewports (e.g., landscape)
2019-12-31 18:11:37 -08:00
b5467bb81c
Don't shrink margins for wider viewports (e.g., landscape)
2019-12-31 20:08:40 -06:00
dcb6f06792
Merge pull request #1504 from Wevah/ios-blockquote-tweaks
...
Tweak blockquote margins for small viewports
2019-12-31 17:56:14 -08:00
95db828986
Tweak blockquote margins for small viewports
2019-12-31 19:47:09 -06:00
e4c84bc501
Add two missing expectation fulfillments
...
It looks like two tests in FeedlySetStarredArticlesOperationTests
created but never referenced XCTestExpectation instances. Based on the
other nearby tests, add a call to `fulfill()` inside the associated
completion block after the rest of our test assertions are done.
2019-12-31 19:41:32 -06:00
6c10774c4a
Fix build errors stemming from FeedlyTestSupport
...
There was one call to a throwing function inside
`checkArticles(in:correspondToStreamItemsIn:)` which was not
appropriately marked with `try`. Add that keyword, and then bubble out
the chain of errors through additional layers of helpers to the
enclosing test:
* This `checkArticles` variant was called by two others
* …one of which was used in `testAddNewFeedSuccess()`
* …another of which was used in various `verify` sync helpers
* …which were referenced from `testSyncing()`, a test case method
None of these involved any particular async hoops to jump through, and
since the top-level callers were all test functions, we can count on
XCTest to handle any errors thrown — no additional `catch` or handling
on our part is necessary.
2019-12-31 19:36:42 -06:00
711aca3d1b
Fix build errors in Feedly test support
...
Two more cases of completion blocks taking Results, requiring a
do/catch/Result.get() to unwrap.
This commit deliberately leaves one build error for a more comprehensive
fix, since it occurs in a helper function that will have broader
fallout.
2019-12-31 19:29:44 -06:00
152b1f2b8a
Fix errors in Feedly sync unread operations tests
...
Yet more completion blocks with Results; apply the usual
do/catch/Result.get() dance.
2019-12-31 19:27:47 -06:00
13b227a461
Fix errors in Feedly set starred operations tests
...
More of the same: completion blocks which take Results need
do/catch/Result.get().
2019-12-31 19:24:36 -06:00
f12e8b4a4a
Fix errors in starred articles operations tests
...
Pull the same do/catch/Result.get() trick as previously for completion
blocks which now take Results. `try` a few throwing calls.
2019-12-31 19:11:37 -06:00
6a4f3fb11e
Fix build errors in sync stream operations tests
...
Prefix two throwing calls with `try`. Label the enclosing test functions
with `throws` so the XCTest machinery helps us with error reporting.
2019-12-31 19:08:29 -06:00
c2d298b836
Add the image title text to zoomed images. Issue #1468
2019-12-31 18:06:27 -07:00
6583688e38
Fix errors in Feedly unread operations tests
...
Most fetch completion blocks took a parameter that was expected to be
some result data type, but is now a Result. Rename these parameters;
wrap their existing bodies in do/catch blocks; and recreate the original
underlying variable using the result of `Result.get()`.
Prepend a few synchronous calls that started throwing with `try` along
the way.
2019-12-31 19:06:10 -06:00
0d915333e6
Merge branch 'ios-candidate' of https://github.com/brentsimmons/NetNewsWire into ios-candidate
2019-12-31 17:06:32 -07:00
f0a5309786
Fix bug that prevented first article from being selected on iPad.
2019-12-31 17:06:13 -07:00
af76e44c0f
Changed ArticleViewController to utilize UIPageViewController to provide gesture based navigation.
2019-12-31 16:55:39 -07:00
6d3cd0cc19
Update to build 24.
2019-12-31 15:40:31 -08:00
520ef3858a
Merge branch 'ios-candidate' of https://github.com/brentsimmons/NetNewsWire into ios-candidate
2019-12-31 15:40:10 -08:00
51faf77e59
Update isExecuting and isFinished in a way that should make NSOperationQueue happy. Hopefully this fixes a mystery crash bug.
2019-12-31 15:40:05 -08:00
0a29535412
Merge pull request #1496 from Wevah/table-overflow-wrapper
...
Table overflow wrapper
2019-12-31 16:24:43 -07:00
ae1651fad1
Fix errors in Feedly update-feeds operation tests
...
Mark some throwing calls with `try`; mark the enclosing test functions
with `throws`, letting the XCTest framework handle error reporting for
us.
2019-12-31 15:55:59 -06:00
f82be27666
Resolve errors in Feedly statuses operation tests
...
Add `error` parameters to completion blocks which now pass them. Assert
these errors are always nil in the existing tests.
Flip calls to `selectPendingCount()` so they are async, with a
completion block that asserts about the results instead of asserting
about the return value. Since the closure takes a Result, unwrap it in a
do/catch block at each site; `XCTAssertNoThrow` doesn't help us bubble a
value out from `Result.get()`, and I'd rather not use `try!` here. There
might be a stylistic discussion to be had about this unwrapping, though.
2019-12-31 15:52:54 -06:00
73b9d03fac
Merge pull request #1497 from Wevah/allow-size-styles-ios
...
Only strip color/font inline styles
2019-12-31 13:50:37 -08:00
fa6c21f470
Remove .wp-smiley rule
2019-12-30 18:30:21 -06:00
fcea33d61f
Only strip out color, background, and font css properties
...
Prevents icon images from rendering super-huge.
2019-12-30 18:30:10 -06:00
792fcddaf6
Update comment
2019-12-30 17:35:47 -06:00
58ef7600fb
Avoid copying the table and use querySelector()
...
(Since there's only one .articleBody.)
2019-12-30 17:24:34 -06:00
7a61a6a5b4
Specify div explicitly
2019-12-30 17:24:34 -06:00
0800c77002
Remove now-unneeded style rule
2019-12-30 17:17:35 -06:00
c2d3e9213d
Strip spans from pre elements inside table cells
...
Works around a clipping issue.
2019-12-30 17:17:35 -06:00
bb0fffcec2
Move wrapTables() to just after wrapFrames()
2019-12-30 17:17:35 -06:00
cbbb46a776
Wrap tables in an overflow-x: auto; div
2019-12-30 17:17:35 -06:00
8a85b18d09
Merge pull request #1495 from kielgillard/ios-candidate
...
Models the feed identifier of feedly articles as an optional…
2019-12-30 14:55:30 -07:00
3b47edfba7
Models the feed identifier of feedly articles as an optional since it seems the Feedly API will not always provide one (despite the API documentation).
...
Additional, more elegantly handle the failure to parse an article when Feedly does not provide a matching feed.
2019-12-31 08:47:27 +11:00
c692d46cc8
Merge pull request #1493 from hartlco/1486-feedslist-contextmenu-mark-all-as-read
...
Include Mark All as Read in Feeds list context menu #1486
2019-12-30 13:37:38 -07:00
5cbfe3b071
Include Mark All as Read in Feeds list context menu
2019-12-30 21:16:50 +01:00
388886a355
Base our active articles window on dateArrived instead of datePublished. This reverses a recent change that led to some bugs.
2019-12-28 22:20:13 -08:00
ac519c5c90
Add new and updated articles to the in-memory cache when saving articles.
2019-12-28 21:19:02 -08:00
26bb2b8eaf
Remove references to Article.bannerImageURL. We can bring it back if/when we start using it.
2019-12-28 20:53:36 -08:00
a1b23edc8b
Delete obsolete code
2019-12-28 16:06:37 -07:00
f37ce4357e
Delete obsolete code
2019-12-28 16:04:24 -07:00
5c4d8aeba7
Verify that the parent node to be expanded is a Folder before trying to expand it.
2019-12-28 10:58:15 -07:00
0fbd376078
Merge branch 'ios-candidate' of https://github.com/brentsimmons/NetNewsWire into ios-candidate
2019-12-28 10:23:43 -07:00
36521a1783
Don't crash if there isn't a folder or feed available.
2019-12-28 10:23:09 -07:00
aeaeac61fc
Get rid of DatabaseArticle entirely. Cache Article objects. This will make fetches faster *and* save memory.
2019-12-27 22:47:02 -08:00
4b361300f4
Fix the OPML export when there is only one exportable account. Issue #1457
2019-12-26 12:35:39 -07:00
1043b42472
Merge branch 'ios-candidate' of https://github.com/brentsimmons/NetNewsWire into ios-candidate
2019-12-26 12:22:10 -07:00
00ed368683
Change the full screen tap zone to be the center of the navigation bar instead of the whole navigation bar. Issue #1450 .
2019-12-26 12:21:56 -07:00
Brent Simmons
Nate Weaver
Tim Ekl
Maurice Parker
Kiel Gillard
Martin Hartl