Merge pull request #815 from lbryio/channel-name-page

Highlight channel names in pages.
This commit is contained in:
hackrush 2017-12-07 00:39:32 +05:30 committed by GitHub
commit 8f83337640
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 12 additions and 4 deletions

View file

@ -14,6 +14,8 @@ Web UI version numbers should always match the corresponding version of LBRY App
### Changed ### Changed
* The credit balance displayed in the main app navigation displays two decimal places instead of one. * The credit balance displayed in the main app navigation displays two decimal places instead of one.
* Moved all redux code into /redux folder * Moved all redux code into /redux folder
* Channel names in pages are highlighted to indicate them being clickable(#814)
*
### Fixed ### Fixed
* Long channel names causing inconsistent thumbnail sizes (#721) * Long channel names causing inconsistent thumbnail sizes (#721)

View file

@ -1,7 +1,8 @@
import React from "react"; import React from "react";
import { Icon } from "component/common"; import { Icon } from "component/common";
import Link from "component/link"; import Link from "component/link";
import lbryuri from "lbryuri.js"; import lbryuri from "lbryuri";
import classnames from "classnames";
class UriIndicator extends React.PureComponent { class UriIndicator extends React.PureComponent {
componentWillMount() { componentWillMount() {
@ -60,7 +61,12 @@ class UriIndicator extends React.PureComponent {
const inner = ( const inner = (
<span> <span>
<span className={`channel-name ${smallCard && "channel-name--small"}`}> <span
className={classnames("channel-name", {
"channel-name--small": smallCard,
"button-text no-underline": link,
})}
>
{channelName} {channelName}
</span>{" "} </span>{" "}
{!signatureIsValid ? ( {!signatureIsValid ? (

View file

@ -1,9 +1,9 @@
.channel-name { .channel-name {
display: inline-block; display: inline-flex;
overflow: hidden; overflow: hidden;
white-space: nowrap; white-space: nowrap;
text-overflow: ellipsis text-overflow: ellipsis;
} }
// this shouldn't know about the card width // this shouldn't know about the card width