Commit graph

1928 commits

Author SHA1 Message Date
zeppi 2d66b8666d styling 2021-08-03 11:05:32 -04:00
saltrafael f01fb7dd22 Mobile view: Make comments expandable 2021-08-03 11:05:32 -04:00
infinite-persistence 1d6101ddd2 Report recsys info when 'share_data_usage' is enabled.
Previous:
~~Although it was collecting info for all platforms, the data is only sent for Odysee.~~ Correction: it became enabled for all platform after `master` became Odysee.

New:
Only send out the telemetry for `IS_WEB` or if there's user consent.
2021-08-03 09:34:36 -04:00
infinite-persistence 34368760de Fill in remaining Recsys fields
## Issue
6366 Recsys Evaluation Telemetry

The recommended list from lighthouse is obtained from `makeSelectRecommendedContentForUri`. This list is further tweaked by the GUI (e.g. move autoplay next item to top, remove blocked content, etc.). Recsys wants the final recommendation list and the clicked index (in exact order), so we need pass these info to the videojs recsys plugin somehow. Also, Recsys wants a recommendation list ID as well as the parent (referrer) ID, we so need to track the clicks and navigation.

## General Approach
- It seems easiest to just spew back the final (displayed) list and all the required info to Redux, and the recsys plugin (or anyone else in the future) can grab it.
- Try to touch few files as possible. The dirty work should all reside in `<RecommendedContent>` only.

## Changes
- `ClaimPreview`: add optional parameters to store an ID of the container that it is in (for this case, it is `ClaimList`) as well as the index within the container.
  - When clicked, we store the container ID in the navigation history `state` object.
    - For general cases, anyone can check this state from  `history.location.state` to know which container referred/navigated to the current page. For the recsys use case, we can use this as the `parentUUID`.
- `ClaimList`: just relay `onClick` and set IDs.
- `RecommendedContent`: now handles the uuid generation (for both parent and child) and stores the data in Redux.
2021-08-03 09:34:36 -04:00
infinite-persistence 74659f40a1
Restore "Use better icon for "Copy Link"" -- lost in a merge.
b164a5d1 accidentally nulled the changed.

While at it, moved the "Copy RSS" button below "Copy Link" -- I think that looks better.
2021-08-03 14:24:02 +08:00
zeppi 26b1c901c8 no limit on stream types for following 2021-08-02 16:14:33 -04:00
btzr-io c0c65c46f4 use env for twitter account 2021-08-01 09:39:16 -04:00
btzr-io d462f96834 improve twitter share 2021-08-01 09:39:16 -04:00
Baltazar Gomez 37d9cbfbeb limit claim_search results on livestreamLink 2021-08-01 09:13:25 -04:00
btzr-io 9983777f86 faster claim search on livestreamLink 2021-08-01 09:13:25 -04:00
btzr-io 7802337655 prevent livestream api calls if no livestream claim exists: #6632 2021-08-01 09:13:25 -04:00
Thomas Zarebczan b5fec393c6
Fix crash 2021-07-31 11:08:15 -04:00
zeppi b93caf4ebc fix logo image 2021-07-30 12:12:58 -04:00
saltrafael 87ae041472 Add confirmation on comment removal 2021-07-30 10:24:04 -04:00
Baltazar Gomez 415a522805 fix notification menu button styles 2021-07-30 00:11:18 -07:00
infinite-persistence 0cdf881941
#6470 Improve accessibility and some minor css fixes 2021-07-30 09:34:40 +08:00
saltrafael cc93ac1a01 Fix currently active playing item on playlist 2021-07-29 14:55:52 -04:00
infinite-persistence 7537b5ae7b
Lint and cleanup. No functional change. 2021-07-29 21:11:20 +08:00
infinite-persistence 4ff32e0c61
Fix missing file-thumbnail in Notifications
## Issue
6721 Notifications: Thumbnail for "new content" not showing up in Desktop layout

## Change
The `className` wasn't propagated during the `FileThumbnail` refactoring to `Thumbs`.
2021-07-29 15:56:42 +08:00
Baltazar Gomez 74087d2b86
Merge branch 'master' into accessibility 2021-07-28 23:47:42 -05:00
zeppi 9889c258e3 livestream api constant rename 2021-07-28 16:51:11 -04:00
zeppi d25896d99c recon commentCreate 2021-07-28 15:43:56 -04:00
zeppi edebde5a2e file reactions recon 2021-07-28 08:12:29 -04:00
zeppi 2a69c42e5b back to img tag 2021-07-27 15:13:29 -04:00
zeppi b1e8d2d6db desktop logo 2021-07-27 15:13:29 -04:00
zeppi ef77a5cd00 optimizedImage 2021-07-27 15:13:29 -04:00
zeppi 71d9b0d973 default logo 2021-07-27 15:13:29 -04:00
zeppi 3f4c601428 logo component on embeds/embeds ended 2021-07-27 15:13:29 -04:00
zeppi 19f8b0adac logoComponent 2021-07-27 15:13:29 -04:00
zeppi e4d7e4f69d force has source channel page 2021-07-26 14:17:52 -04:00
zeppi 403ea8f6f4 no livestreams on channel content 2021-07-26 14:17:52 -04:00
infinite-persistence 6de16813cb
i18n update 2021-07-26 23:31:03 +08:00
infinite-persistence 8dc18e8fcd
Fix split (concat) string usage
Concatenated strings are hard to translate.
2021-07-26 12:37:59 +08:00
zeppi 0987c6e94f publish form recon 2021-07-23 10:14:22 -04:00
infinite-persistence ffad52d5b7 Remove console.log
Seems like lint is being bypassed recently?
2021-07-23 08:42:58 -04:00
infinite-persistence 7a8d213861
Eliminate CLS in Recommended due to wrapper mismatch
Seem like `type === small` was missed out, so it was using a wrapper than necessary, causing the shift.
2021-07-23 13:23:52 +08:00
zeppi 038431f95c channel follow first run on master 2021-07-23 00:00:32 -04:00
zeppi ed0387d94a pinning 2021-07-22 23:24:55 -04:00
zeppi d88752c646 music icon, fix lint 2021-07-22 17:31:46 -04:00
Anthony 78b4fa05b3 pull out variable for max and min fiat amounts 2021-07-22 14:06:59 -04:00
Anthony 7975ccf3e0 disable button if theres a fiat tip error 2021-07-22 14:06:59 -04:00
Anthony 713dc69ed7 add check for two decimals and fix showing error for fiat tip 2021-07-22 14:06:59 -04:00
zeppi f4559c1cce fileSubtitle recon 2021-07-22 12:09:54 -04:00
zeppi 8919182c0c default avatar config 2021-07-22 09:44:57 -04:00
btzr-io 86e1cfc3dd refactoring of claim preview aria-label format 2021-07-22 03:24:25 -05:00
btzr-io 2a291366da use full sentences with variables for claim preview aria-label 2021-07-22 02:37:57 -05:00
infinite-persistence 032e17cecd
Improve CLS on Posts
## Issue
6068 Fix "Cumulative Layout Shift" for Core Web Vitals

In Posts, the Comments section appears first while we fetch the MD file. When the MD is fetched, Comments get pushed to the bottom (or shifted up for short posts), hence the red CLS scores.

## Approach
There are too many layers between `<FilePage>` and `<DocumentViewer>` to pass the `loading` state around to hide the Comments section, so just make Comments fade in after a 2s delay.

## Changes
- Posts: Add 2s fade-in delay to Comments.
- Posts: remove the gray placeholder. This improves the score a bit more, and reduces flicker as well. There's already a spinner from `FileRenderInline` to tell the user to be patient.
- Posts: add a minimum 30vh height so that short posts don't get collapsed too much, causing the `FileDetails` and Comments to shift. Small shifts are fine as long as CLS is below 0.1.
2021-07-22 10:38:01 +08:00
Baltazar Gomez c983af927f
Merge branch 'master' into accessibility 2021-07-21 20:59:12 -05:00
infinite-persistence 50c606d916 claimListDiscover: don't re-render until query is done
## Issue
6542 Livestream listing and blocked list kills CLS score

This is basically a repeat of what was done on `claimTileDiscover` (homepage), but now on `claimListDiscover` (category pages).

This handles the unnecessary re-render when:
- the uri list temporarily being zero while waiting for new claim-search results.
- livestream claim-search invalidating the list.

## Changes
Store the last uri list and use that when we know that claim-search is still not done.

## Tests
1. Ran Lighthouse on category pages (force a livestream channel on it). CLS score must be green.
2. Block `claim_search` from Dev Tools and move to another category. The page should say "timed out" instead of "no results" (i.e. maintain existing behavior).
2021-07-21 21:51:12 -04:00
zeppi 1994b5da04 guidelines optional in sidenav 2021-07-21 20:26:17 -04:00