Skip to content

Commit

Permalink
[Look&Feel] Consistency of Plus Icons (#7195)
Browse files Browse the repository at this point in the history
* Updated plus icon over plusInCircle for add/create use cases.

Signed-off-by: Dan Dong <danieldong51@gmail.com>

* Added plus icon to dashboard creation.

Signed-off-by: Dan Dong <danieldong51@gmail.com>

* Reverted Dashboard Creation Icon

Signed-off-by: Dan Dong <danieldong51@gmail.com>

* Changeset file for PR #7195 created/updated

* Revert plus icon changes for add use cases

Signed-off-by: Dan Dong <danieldong51@gmail.com>

* Fixed snapshot testing

Signed-off-by: Dan Dong <danieldong51@gmail.com>

---------

Signed-off-by: Dan Dong <danieldong51@gmail.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
  • Loading branch information
1 parent b422791 commit a2ee362
Show file tree
Hide file tree
Showing 13 changed files with 25 additions and 28 deletions.
2 changes: 2 additions & 0 deletions changelogs/fragments/7195.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
refactor:
- [Look&Feel] Consistency of Plus Icons ([#7195](/~https://github.com/opensearch-project/OpenSearch-Dashboards/pull/7195))

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -78,12 +78,7 @@ const CreateButton = (props: CreateButtonProps) => {
const provider: DashboardProvider = Object.values(props.dashboardProviders!)[0];
return (
<EuiFlexItem grow={false}>
<EuiButton
href={provider.createUrl}
data-test-subj="newItemButton"
iconType="plusInCircle"
fill
>
<EuiButton href={provider.createUrl} data-test-subj="newItemButton" iconType="plus" fill>
<FormattedMessage id="dashboard.listing.createButtonText" defaultMessage="Create" />
&nbsp;{provider.createLinkText}
</EuiButton>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ function getAddConfig(action: NavAction) {
function getCreateNewConfig(action: NavAction) {
return {
emphasize: true,
iconType: 'plusInCircleFilled',
iconType: 'plus',
id: 'addNew',
label: i18n.translate('dashboard.topNave.addNewButtonAriaLabel', {
defaultMessage: 'Create new',
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ export function DashboardEmptyScreen({
<EuiButton
iconSide="left"
size="s"
iconType="plusInCircle"
iconType="plus"
onClick={onVisualizeClick}
data-test-subj="addVisualizationButton"
aria-label={constants.createNewVisualizationButtonAriaLabel}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ export const getNoItemsMessage = (
<EuiButton
onClick={createItem}
fill
iconType="plusInCircle"
iconType="plus"
data-test-subj="createDashboardPromptButton"
>
<FormattedMessage
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ export function SavedObjectFinderCreateNew({ menuItems }: Props) {
button={
<EuiButton
data-test-subj="createNew"
iconType="plusInCircle"
iconType="plus"
iconSide="left"
onClick={toggleCreateMenu}
fill
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ export class CreateButton extends Component<Props, State> {
data-test-subj="createIndexPatternButton"
fill={true}
onClick={options[0].onClick}
iconType="plusInCircle"
iconType="plus"
>
{children}
</EuiButton>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -505,7 +505,7 @@ class TableListView extends React.Component<TableListViewProps, TableListViewSta
<EuiButton
onClick={this.props.createItem}
data-test-subj="newItemButton"
iconType="plusInCircle"
iconType="plus"
fill
>
<FormattedMessage
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,7 @@ function DateRangesParamEditor({
<EuiSpacer size="s" />
<EuiFlexItem>
<EuiButtonEmpty
iconType="plusInCircleFilled"
iconType="plus"
onClick={onAddRange}
size="xs"
data-test-subj="visEditorAddDateRange"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ export const getNoItemsMessage = (createItem: () => void) => (
<EuiButton
onClick={createItem}
fill
iconType="plusInCircle"
iconType="plus"
data-test-subj="createVisualizationPromptButton"
>
<FormattedMessage
Expand Down

0 comments on commit a2ee362

Please sign in to comment.