Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Readup back on the web #59

Merged
merged 49 commits into from
Mar 12, 2023
Merged

Readup back on the web #59

merged 49 commits into from
Mar 12, 2023

Conversation

th0rgall
Copy link
Member

@th0rgall th0rgall commented Oct 7, 2022 •
edited
Loading

Addresses issue #50.

th0rgall and others added 30 commits October 7, 2022 21:54
@th0rgall
Revert the extension source code to commit faadeb7
48e24c8
@th0rgall
Attempt to re-integrate the old extension and BrowserRoot with log-in
0a5b04b
@th0rgall
Revert "only include credentials for app clients"
4c7e2ea 
This reverts commit 1dd6653.
@th0rgall
Add some docs
a7ed2ed
@th0rgall
Add minor doc comments
945d986
@th0rgall
Merge branch 'master' into extension-as-proxy
65c22dc
@th0rgall
Add some documentation of the simulated reader
1024179
@th0rgall
Some comment changes and condensed promise chain with async/await for…
6808377 
… readability
@th0rgall
Merge branch 'extension-as-proxy' of github.com:reallyreadit/web into…
6b867d1 
… extension-as-proxy
@th0rgall
Add some docs & cleanup
9b5f0ee
@th0rgall
Add App.tsx for the reader page
c1dda73
@th0rgall
Some docs in the native reader
07f42de
@th0rgall
Wip: start converting content script to the "extension reader" page
ec12af7
@th0rgall
Wip: refactor the "extension reader" to use the native App.tsx reader UI
85f52bc
@th0rgall
Refactor App.tsx into a common ReaderUIEmbed.tsx
331e0b2
@th0rgall
Implement initial readArticle & nav back
89164a9
@th0rgall
Various style fixes using inline style loading, removal of unnecessar…
a95cc0a 
…y content parser build
@th0rgall
Delete unused component hosts
8434574
@th0rgall
Filename typo fix
48bde99
@th0rgall
Initial article processing & debugging multiple-tab messaging
e13e678
@jacamera
prelim fix for crosstalk between reader scripts
48ed488
@th0rgall
More accurate relative image processor (not fully tested)
46083af
@th0rgall
Restoring the original BrowserRoot UI
952030a
@th0rgall
Build-scripts to produce a separate mv3 extension
c540e3c
@th0rgall
Copy paste of event-page/main.ts into service-worker.ts
533c3dc
@th0rgall
Initial mv3 migration changes
2344471
@th0rgall
Add AsyncObjectStore and AsyncSetStore backed by chrome.storage
a53242a 
Replaces synchronous stores backed by window.localStorage
+ misc formatting changes, small fixes
@th0rgall
Fix jumpy loading & theme blinks,
032d63a 
start attempt of static declarativeNetRequest rules
@th0rgall
Fix MV2 after changes, test in Firefox
d47487e
@th0rgall
Merge branch 'master' into extension-as-proxy-mv3
d08c908
@th0rgall
fix: web_accessible_resources for the alert content script in mv3
3e22ea1
@th0rgall
feat: add "navigate back" behavior when clicking the action button wh…
f920b2e 
…ile in reader mode
@th0rgall
fix: open read links in reader article comments in the reader
5a78ce3 
Also a slight refactor in ReaderContentScriptApi link handling.
@th0rgall
style: format with Rome
8f9f069
@th0rgall
style: format with Prettier (scss,css,json,html,md)
52e7d5d 
Match .editorconfig settings.
@th0rgall
style: remove dummy test file
b095a10
@th0rgall
feat: share article source url, open in web reader on /read/ link
e3e48c1
@th0rgall
style: remove MarketingBanner
b2215e8
@th0rgall
fix: update download/get started prompts for the web app
a7aa4b7 
Also adds a like for contributions in the web app.
@th0rgall
fix: whole tab is undefined on unregisterPage, not just tab.id
6f4128c
@th0rgall
style: rome formatting suggestions
276dad1
@th0rgall
fix: restore the Options page for "star on save" (open)
2e501d0
@th0rgall
build: add extensions to workspace recommendations
ce18fce
@th0rgall
fix: prevent a flash of the AlertBadge when badge is undefined/null
f911f84
@th0rgall
Merge pull request #60 from reallyreadit/extension-as-proxy-mv3
ef7211d 
Manifest v3 extension migration
@th0rgall th0rgall marked this pull request as ready for review March 4, 2023 14:09
jacamera
jacamera approved these changes Mar 12, 2023
View reviewed changes
Copy link
Member

@jacamera jacamera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working great in Chrome and Firefox! 🎉

@jacamera jacamera merged commit e79ea5f into master Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jacamera jacamera jacamera approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@th0rgall @jacamera