FIx Bug 60425: Add highlighting for user's email
This commit is contained in:
parent
36298e8a48
commit
f8fc8343f5
@ -389,6 +389,7 @@ const PeopleTableRow = (props) => {
|
|||||||
color={sideInfoColor}
|
color={sideInfoColor}
|
||||||
onClick={onEmailClick}
|
onClick={onEmailClick}
|
||||||
isTextOverflow
|
isTextOverflow
|
||||||
|
enableUserSelect
|
||||||
>
|
>
|
||||||
{email}
|
{email}
|
||||||
</Link>
|
</Link>
|
||||||
|
@ -3,7 +3,13 @@ import PropTypes from "prop-types";
|
|||||||
import StyledText from "./styled-link";
|
import StyledText from "./styled-link";
|
||||||
|
|
||||||
// eslint-disable-next-line react/display-name
|
// eslint-disable-next-line react/display-name
|
||||||
const Link = ({ isTextOverflow, children, noHover, ...rest }) => {
|
const Link = ({
|
||||||
|
isTextOverflow,
|
||||||
|
children,
|
||||||
|
noHover,
|
||||||
|
enableUserSelect,
|
||||||
|
...rest
|
||||||
|
}) => {
|
||||||
// console.log("Link render", rest);
|
// console.log("Link render", rest);
|
||||||
|
|
||||||
return (
|
return (
|
||||||
@ -12,6 +18,7 @@ const Link = ({ isTextOverflow, children, noHover, ...rest }) => {
|
|||||||
isTextOverflow={isTextOverflow}
|
isTextOverflow={isTextOverflow}
|
||||||
noHover={noHover}
|
noHover={noHover}
|
||||||
truncate={isTextOverflow}
|
truncate={isTextOverflow}
|
||||||
|
enableUserSelect={enableUserSelect}
|
||||||
{...rest}
|
{...rest}
|
||||||
>
|
>
|
||||||
{children}
|
{children}
|
||||||
@ -57,6 +64,8 @@ Link.propTypes = {
|
|||||||
title: PropTypes.string,
|
title: PropTypes.string,
|
||||||
/** Type of link */
|
/** Type of link */
|
||||||
type: PropTypes.oneOf(["action", "page"]),
|
type: PropTypes.oneOf(["action", "page"]),
|
||||||
|
|
||||||
|
enableUserSelect: PropTypes.bool,
|
||||||
};
|
};
|
||||||
|
|
||||||
Link.defaultProps = {
|
Link.defaultProps = {
|
||||||
@ -71,6 +80,7 @@ Link.defaultProps = {
|
|||||||
rel: "noopener noreferrer",
|
rel: "noopener noreferrer",
|
||||||
tabIndex: -1,
|
tabIndex: -1,
|
||||||
type: "page",
|
type: "page",
|
||||||
|
enableUserSelect: false,
|
||||||
};
|
};
|
||||||
|
|
||||||
export default Link;
|
export default Link;
|
||||||
|
@ -22,7 +22,7 @@ const PureText = ({ type, color, ...props }) => <Text {...props} />;
|
|||||||
const StyledText = styled(PureText)`
|
const StyledText = styled(PureText)`
|
||||||
text-decoration: ${(props) => props.theme.link.textDecoration};
|
text-decoration: ${(props) => props.theme.link.textDecoration};
|
||||||
|
|
||||||
${NoUserSelect}
|
${(props) => !props.enableUserSelect && NoUserSelect}
|
||||||
|
|
||||||
cursor: ${(props) => props.theme.link.cursor};
|
cursor: ${(props) => props.theme.link.cursor};
|
||||||
-webkit-tap-highlight-color: rgba(0, 0, 0, 0);
|
-webkit-tap-highlight-color: rgba(0, 0, 0, 0);
|
||||||
|
Loading…
Reference in New Issue
Block a user