File viewer v2 #1826
Labels
No labels
accessibility
app-parity
area: creator
area: daemon
area: design
area: devops
area: discovery
area: docs
area: installer
area: internal
area: livestream
area: performance
area: proposal
area: reposts
area: rewards
area: search
area: security
area: subscriptions
area: sync
area: ux
area: viewer
area: wallet
BEAMER
channel
comments
community PR
consider soon
core team
css
dependencies
electron
Epic
feature request
first-timers-only
good first issue
hacktoberfest
help wanted
hub-dependent
icebox
Invalid
level: 0
level: 1
level: 2
level: 3
level: 4
merge when green
needs: exploration
needs: grooming
needs: priority
needs: repro
needs: tech design
notifications
odysee
on hold
playlists
priority: blocker
priority: high
priority: low
priority: medium
protocol dependent
recsys
redesign
regression
resilience
sdk dependent
Tom's Wishlist
trending
type: bug
type: discussion
type: improvement
type: new feature
type: refactor
type: task
type: testing
unplanned
windows
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: LBRYCommunity/lbry-desktop#1826
Loading…
Reference in a new issue
No description provided.
Delete branch "fix-viewer"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Changes (WIP)
77cfcc84a8
.CodeMirror
context menube9c16cbde
.CodeMirror
theme (light / dark).Test
Looks good, nice work!
I'll need to take a closer look and test this out; @seanyesmunt also had some MD changes as well that I need to test and check for conflicts.
https://github.com/codemirror/CodeMirror/issues/5484
FYI @btzr-io @seanyesmunt is on vacation until Wednesday
Will test this today
Just added this to test lbry://javascriptfile#c286d02c447aecb9cbd83b6f3ee6811e8fcc3d68
This is so cool. 🙂
One minor comment. Can you add a changelog too? Then it's ready to be merged.
padding in this file should use the spacing variables
Whoops forgot to add this comment. This
z-index
seems a little weird. Is it needed?not sure let me check, probably not 👍
@seanyesmunt ok done 👍
On lbry://@jiggytom#e48d2b50501159034f68d53321f67b8aa5b1d771/epubfile - a non-supported file, I'm missing a view button. Not sure if it should still say view, but lbry://zipfile does and it's an unsupported file. This isn't related to this PR, but thought i'd mention it as we are touching that code.
Otherwise, I tested a bunch of the videos and things look good! Thanks again for the contribution @btzr-io !