Unverified Commit cf8cbc92 authored by Andrey Azov's avatar Andrey Azov Committed by GitHub
Browse files

Remove condition upon environment from ZmenuContend (when showing links) (#150)

parent b4fc297d
Pipeline #32573 passed with stages
in 4 minutes and 16 seconds
...@@ -2,8 +2,6 @@ import React from 'react'; ...@@ -2,8 +2,6 @@ import React from 'react';
import classNames from 'classnames'; import classNames from 'classnames';
import { connect } from 'react-redux'; import { connect } from 'react-redux';
import { isEnvironment, Environment } from 'src/shared/helpers/environment';
import { changeFocusObject } from 'src/content/app/browser/browserActions'; import { changeFocusObject } from 'src/content/app/browser/browserActions';
import styles from './Zmenu.scss'; import styles from './Zmenu.scss';
...@@ -85,16 +83,11 @@ const ZmenuContentBlock = (props: ZmenuContentBlockProps) => { ...@@ -85,16 +83,11 @@ const ZmenuContentBlock = (props: ZmenuContentBlockProps) => {
const ZmenuContentItem = (props: ZmenuContentItemProps) => { const ZmenuContentItem = (props: ZmenuContentItemProps) => {
const { text, markup, id } = props; const { text, markup, id } = props;
// FIXME: remove dependency on environment when ensObject api endpoint is ready
const shouldAddLink =
markup.includes(Markup.FOCUS) &&
isEnvironment([Environment.DEVELOPMENT, Environment.INTERNAL]);
const className = classNames({ const className = classNames({
[styles.markupLight]: markup.includes(Markup.LIGHT), [styles.markupLight]: markup.includes(Markup.LIGHT),
[styles.markupStrong]: markup.includes(Markup.STRONG), [styles.markupStrong]: markup.includes(Markup.STRONG),
[styles.markupEmphasis]: markup.includes(Markup.EMPHASIS), [styles.markupEmphasis]: markup.includes(Markup.EMPHASIS),
[styles.markupFocus]: shouldAddLink [styles.markupFocus]: markup.includes(Markup.FOCUS)
}); });
const handleClick = () => { const handleClick = () => {
...@@ -103,7 +96,7 @@ const ZmenuContentItem = (props: ZmenuContentItemProps) => { ...@@ -103,7 +96,7 @@ const ZmenuContentItem = (props: ZmenuContentItemProps) => {
const itemProps = { const itemProps = {
className, className,
onClick: shouldAddLink ? handleClick : undefined onClick: handleClick
}; };
return <span {...itemProps}>{text}</span>; return <span {...itemProps}>{text}</span>;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment