File viewer v2 #1826

Merged
btzr-io merged 17 commits from fix-viewer into master 2018-08-06 03:53:57 +02:00
btzr-io commented 2018-07-27 02:32:06 +02:00 (Migrated from github.com)

Changes (WIP)

  • Closes: Better preview for content #620
  • Closes: Use router pattern for rendering file viewer (#1544)
  • Temp solution for: Add csv and json viewer (#1410)
  • Markdown viewer. (md, markdown)
  • Plain-text files and code snippets viewer (txt, json, csv, xml, log, yml).
  • Docx viewer (text only)
  • Html viewer. (iframe sandbox)
  • Implement syntax highlighter.
  • Fix description field fullscreen styles 77cfcc84a8.
  • Fix CodeMirror context menu be9c16cbde.
  • Choose a CodeMirror theme (light / dark).
  • Refactoring of fileSource logic.
  • Problem with search auto-complete menu when scrolling over file viewer #1847

Test

lbry://docxfile#a035152583aaa2ed50e6ab12174839a2cc2c0227

lbry://pdffile#1ec33dbcdf61c051b136354a9640f5dc25b04b96

lbry://csvfile#3eb2ae7de726f21318fc61c6794960f7fc4d81da

lbry://json#5fd9811240848eeae120166d8c57fc8e81dc6497

lbry://txtfile#13475689e84cd6c3ef43a279c0e8bfa6ed5c312f

lbry://index#6e477eded9468613c9c5b45eb58662ea302bccea

lbry://mdfile#24e737e03df62930ac4bd44151c818b3933a0381

bry://javascriptfile#c286d02c447aecb9cbd83b6f3ee6811e8fcc3d68

### Changes (WIP) - [x] Closes: Better preview for content #620 - [x] Closes: Use router pattern for rendering file viewer ([#1544](https://github.com/lbryio/lbry-desktop/pull/1544)) - [x] Temp solution for: Add csv and json viewer ([#1410](https://github.com/lbryio/lbry-desktop/pull/1410)) - [x] Markdown viewer. (md, markdown) - [x] Plain-text files and code snippets viewer (txt, json, csv, xml, log, yml). - [x] Docx viewer (text only) - [x] Html viewer. (iframe sandbox) - [x] Implement syntax highlighter. - [x] Fix description field fullscreen styles https://github.com/lbryio/lbry-desktop/pull/1826/commits/77cfcc84a81f1e629895d7f2019a318d1a11de30. - [x] Fix `CodeMirror` context menu https://github.com/lbryio/lbry-desktop/pull/1671/commits/be9c16cbdec789b557da118b1fc09b950be7171a. - [x] Choose a `CodeMirror` theme (light / dark). - [x] Refactoring of fileSource logic. - [x] Problem with search auto-complete menu when scrolling over file viewer #1847 ### Test > lbry://docxfile#a035152583aaa2ed50e6ab12174839a2cc2c0227 > lbry://pdffile#1ec33dbcdf61c051b136354a9640f5dc25b04b96 > lbry://csvfile#3eb2ae7de726f21318fc61c6794960f7fc4d81da > lbry://json#5fd9811240848eeae120166d8c57fc8e81dc6497 > lbry://txtfile#13475689e84cd6c3ef43a279c0e8bfa6ed5c312f > lbry://index#6e477eded9468613c9c5b45eb58662ea302bccea > lbry://mdfile#24e737e03df62930ac4bd44151c818b3933a0381 > bry://javascriptfile#c286d02c447aecb9cbd83b6f3ee6811e8fcc3d68
skhameneh (Migrated from github.com) reviewed 2018-07-27 02:32:06 +02:00
skhameneh commented 2018-07-27 03:57:43 +02:00 (Migrated from github.com)

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.

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.
btzr-io commented 2018-07-28 01:54:10 +02:00 (Migrated from github.com)
https://github.com/codemirror/CodeMirror/issues/5484
kauffj commented 2018-07-30 18:38:27 +02:00 (Migrated from github.com)

FYI @btzr-io @seanyesmunt is on vacation until Wednesday

FYI @btzr-io @seanyesmunt is on vacation until Wednesday
neb-b commented 2018-08-03 16:50:05 +02:00 (Migrated from github.com)

Will test this today

Will test this today
neb-b commented 2018-08-03 20:04:15 +02:00 (Migrated from github.com)

Just added this to test lbry://javascriptfile#c286d02c447aecb9cbd83b6f3ee6811e8fcc3d68

Just added this to test lbry://javascriptfile#c286d02c447aecb9cbd83b6f3ee6811e8fcc3d68
neb-b (Migrated from github.com) requested changes 2018-08-03 20:04:43 +02:00
neb-b (Migrated from github.com) left a comment

This is so cool. 🙂

One minor comment. Can you add a changelog too? Then it's ready to be merged.

This is so cool. :slightly_smiling_face: One minor comment. Can you add a changelog too? Then it's ready to be merged.
neb-b (Migrated from github.com) commented 2018-08-03 19:41:23 +02:00

padding in this file should use the spacing variables

padding in this file should use the spacing variables
neb-b (Migrated from github.com) reviewed 2018-08-03 20:05:54 +02:00
neb-b (Migrated from github.com) commented 2018-08-03 20:05:54 +02:00

Whoops forgot to add this comment. This z-index seems a little weird. Is it needed?

Whoops forgot to add this comment. This `z-index` seems a little weird. Is it needed?
btzr-io (Migrated from github.com) reviewed 2018-08-03 20:16:51 +02:00
btzr-io (Migrated from github.com) commented 2018-08-03 20:16:51 +02:00

not sure let me check, probably not 👍

not sure let me check, probably not :+1:
btzr-io commented 2018-08-03 20:59:20 +02:00 (Migrated from github.com)

@seanyesmunt ok done 👍

@seanyesmunt ok done :+1:
tzarebczan commented 2018-08-03 23:01:51 +02:00 (Migrated from github.com)

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 !

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 !
neb-b (Migrated from github.com) approved these changes 2018-08-06 03:53:49 +02:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#1826
No description provided.