Clickable channel names on Explore page headings #1820

Closed
opened 2018-07-25 23:09:14 +02:00 by tzarebczan · 2 comments
tzarebczan commented 2018-07-25 23:09:14 +02:00 (Migrated from github.com)

The Issue

Breaking out this issue from https://github.com/lbryio/lbry-desktop/issues/1717.

It would be nice if the channel name in the heading would be a hyperlink to the channel page. With this change, we can have internal APIs pass the long URL, and cut it off after the # for display purposes.

image

Current Sample data:

  "data": {
    "Uris": {
      "Featured Content - DOTA 2 Divine | @Dota2Divine": [
        "miracle-ez-rampage-huskar-free-dmg-bonus#0703b1556ecc1cfccd66f2a1c3e89857336e05ec",
        "badman-spectre-dota-2-big-boss-shutdown#3a69f4d9e6d57e3f02dcac4f5402dfdb40323b1a",
...

To support this change:

  "data": {
    "Uris": {
      "Featured Content - DOTA 2 Divine | @Dota2Divine#2d2c1a3dc98ca868d1f2bfe76b8a1bb820857c5a": [
        "miracle-ez-rampage-huskar-free-dmg-bonus#0703b1556ecc1cfccd66f2a1c3e89857336e05ec",
        "badman-spectre-dota-2-big-boss-shutdown#3a69f4d9e6d57e3f02dcac4f5402dfdb40323b1a",
...

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-desktop/releases - Standalone daemon: https://github.com/lbryio/lbry/releases We are also available on live chat at https://chat.lbry.io --> ## The Issue Breaking out this issue from https://github.com/lbryio/lbry-desktop/issues/1717. It would be nice if the channel name in the heading would be a hyperlink to the channel page. With this change, we can have internal APIs pass the long URL, and cut it off after the `#` for display purposes. ![image](https://user-images.githubusercontent.com/8120721/43227524-f1f314dc-902c-11e8-9e92-988c03cf77e3.png) Current Sample data: ``` "data": { "Uris": { "Featured Content - DOTA 2 Divine | @Dota2Divine": [ "miracle-ez-rampage-huskar-free-dmg-bonus#0703b1556ecc1cfccd66f2a1c3e89857336e05ec", "badman-spectre-dota-2-big-boss-shutdown#3a69f4d9e6d57e3f02dcac4f5402dfdb40323b1a", ... ``` To support this change: ``` "data": { "Uris": { "Featured Content - DOTA 2 Divine | @Dota2Divine#2d2c1a3dc98ca868d1f2bfe76b8a1bb820857c5a": [ "miracle-ez-rampage-huskar-free-dmg-bonus#0703b1556ecc1cfccd66f2a1c3e89857336e05ec", "badman-spectre-dota-2-big-boss-shutdown#3a69f4d9e6d57e3f02dcac4f5402dfdb40323b1a", ... ``` ## 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 -->
chrisza4 commented 2018-10-10 16:33:32 +02:00 (Migrated from github.com)

I just realized I should comment in the issue before taking the issue. Can I take this issue?

I just realized I should comment in the issue before taking the issue. Can I take this issue?
neb-b commented 2018-10-10 16:57:33 +02:00 (Migrated from github.com)

All good. Checking out your PR

All good. Checking out your PR
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#1820
No description provided.