Redesign: NSFW content claim pages not masked/blocked #1546

Closed
opened 2018-06-04 20:06:42 +02:00 by tzarebczan · 3 comments
tzarebczan commented 2018-06-04 20:06:42 +02:00 (Migrated from github.com)

The Issue

In the redesign, you can still navigate to and download properly marked NSFW. The thumbnail does have the red image and note to go into settings on the Discover/channel pages, but you can still click on it to go to the claim page (should not be able to right?) and the claim page allows download/view (should not be able to unless setting is turned).

Sample URL - see homepage or lbry://curl#bcd555ff11ec3084dc23d3b3b70ac7b93e6b3912

System Configuration

  • LBRY Daemon version:
  • LBRY App version:
  • LBRY Installation ID:
  • Operating system:

Anything Else

Screenshots

<!-- Thanks for reporting an issue to LBRY and helping us improve! To make it possible for us to help you, please fill out below information carefully. Before reporting any issues, please make sure that you're using the latest version. - App releases: https://github.com/lbryio/lbry-app/releases - Standalone daemon: https://github.com/lbryio/lbry/releases We are also available on live chat at https://chat.lbry.io --> ## The Issue In the redesign, you can still navigate to and download properly marked NSFW. The thumbnail does have the red image and note to go into settings on the Discover/channel pages, but you can still click on it to go to the claim page (should not be able to right?) and the claim page allows download/view (should not be able to unless setting is turned). Sample URL - see homepage or lbry://curl#bcd555ff11ec3084dc23d3b3b70ac7b93e6b3912 ## System Configuration <!-- For the app, this info is in the About section at the bottom of the Help page. You can include a screenshot instead of typing it out --> <!-- For the daemon, run: curl 'http://localhost:5279' --data '{"method":"version"}' and include the full output --> - LBRY Daemon version: - LBRY App version: - LBRY Installation ID: - Operating system: ## Anything Else <!-- Include anything else that does not fit into the above sections --> ## Screenshots <!-- If a screenshot would help explain the bug, please include one or two here -->
neb-b commented 2018-06-04 20:17:34 +02:00 (Migrated from github.com)

This is intentional. I am very much in favor of allowing people to navigate to the file page. I think the red color and description is enough of a warning.

Interested in hearing others thoughts.

This is intentional. I am very much in favor of allowing people to navigate to the file page. I think the red color and description is enough of a warning. Interested in hearing others thoughts.
tzarebczan commented 2018-06-04 22:27:31 +02:00 (Migrated from github.com)

Ah, now I do remember that conversation. @kauffj do you have a strong stance one way or another?

Ah, now I do remember that conversation. @kauffj do you have a strong stance one way or another?
kauffj commented 2018-06-04 23:27:48 +02:00 (Migrated from github.com)

@tzarebczan I have no opinion about this until my build is working again and I can actually see what we're talking about. I'm also archiving this thread, so please ping me again if you actually want an answer. Also, I trust @seanyesmunt, so we should do whatever he says.

(I do think, in time, it makes sense to evolve filtering from a binary option to a ternary of: on/blur/hide, but that'd be a new feature)

@tzarebczan I have no opinion about this until my build is working again and I can actually see what we're talking about. I'm also archiving this thread, so please ping me again if you actually want an answer. Also, I trust @seanyesmunt, so we should do whatever he says. (I do think, in time, it makes sense to evolve filtering from a binary option to a ternary of: on/blur/hide, but that'd be a new feature)
Sign in to join this conversation.
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#1546
No description provided.