Skip to content

Commit 12eba3a

Browse files
Merge pull request #726 from plotly/sidebar
sidebar tweaks
2 parents 427913e + 3909f79 commit 12eba3a

File tree

6 files changed

+22
-14
lines changed

6 files changed

+22
-14
lines changed

src/DefaultEditor.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,15 +71,17 @@ class DefaultEditor extends Component {
7171
return (
7272
<PanelMenuWrapper>
7373
{logo ? logo : null}
74-
<GraphCreatePanel group={_('Graph')} name={_('Create')} />
75-
<GraphSubplotsPanel group={_('Graph')} name={_('Subplots')} />
76-
{this.hasTransforms() && <GraphTransformsPanel group={_('Graph')} name={_('Transform')} />}
74+
<GraphCreatePanel group={_('Structure')} name={_('Traces')} />
75+
<GraphSubplotsPanel group={_('Structure')} name={_('Subplots')} />
76+
{this.hasTransforms() && (
77+
<GraphTransformsPanel group={_('Structure')} name={_('Transforms')} />
78+
)}
7779
<StyleTracesPanel group={_('Style')} name={_('Traces')} />
7880
<StyleLayoutPanel group={_('Style')} name={_('Layout')} />
7981
{this.hasAxes() && <StyleAxesPanel group={_('Style')} name={_('Axes')} />}
8082
<StyleLegendPanel group={_('Style')} name={_('Legend')} />
8183
{this.hasColorbars() && <StyleColorbarsPanel group={_('Style')} name={_('Color Bars')} />}
82-
<StyleNotesPanel group={_('Style')} name={_('Annotate')} />
84+
<StyleNotesPanel group={_('Style')} name={_('Annotation')} />
8385
<StyleShapesPanel group={_('Style')} name={_('Shapes')} />
8486
<StyleImagesPanel group={_('Style')} name={_('Images')} />
8587
{this.hasSliders() && <StyleSlidersPanel group={_('Style')} name={_('Sliders')} />}

src/components/containers/TraceRequiredPanel.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ class TraceRequiredPanel extends Component {
2222
<PanelEmpty heading={_("Looks like there aren't any traces defined yet.")}>
2323
<p>
2424
{_('Go to the ')}
25-
<a onClick={() => this.context.setPanel('Graph', 'Create')}>{_('Create')}</a>
26-
{_(' panel to define traces.')}
25+
<a onClick={() => this.context.setPanel('Structure', 'Traces')}>{_('Traces')}</a>
26+
{_(' panel under Structure to define traces.')}
2727
</p>
2828
</PanelEmpty>
2929
);

src/components/fields/AxesCreator.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ class UnconnectedAxesCreator extends Component {
152152
{controls}
153153
<Info>
154154
{_('You can style and position your axes in the ')}
155-
<a onClick={() => this.context.setPanel('Graph', 'Subplots')}>{_('Subplots')}</a>
155+
<a onClick={() => this.context.setPanel('Structure', 'Subplots')}>{_('Subplots')}</a>
156156
{_(' panel.')}
157157
</Info>
158158
</PlotlySection>

src/components/fields/SubplotCreator.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ class UnconnectedSubplotCreator extends Component {
133133
/>
134134
<Info>
135135
{_('You can style and position your subplots in the ')}
136-
<a onClick={() => this.context.setPanel('Graph', 'Subplots')}>{_('Subplots')}</a>
136+
<a onClick={() => this.context.setPanel('Structure', 'Subplots')}>{_('Subplots')}</a>
137137
{_(' panel.')}
138138
</Info>
139139
</PlotlySection>

src/default_panels/GraphCreatePanel.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,21 +67,27 @@ const GraphCreatePanel = (props, {localize: _, setPanel}) => {
6767
'Note: in vertical orientation, X values are used for binning. If Y values are provided, they are used as inputs to the histogram function which you can configure in the '
6868
)}
6969
<a onClick={() => setPanel('Style', 'Traces')}>{_('Traces')}</a>
70-
{_(' panel. If Y values are omitted, the histogram function defaults to Count.')}
70+
{_(
71+
' panel under Style. If Y values are omitted, the histogram function defaults to Count.'
72+
)}
7173
</HistogramInfoVertical>
7274
<HistogramInfoHorizontal>
7375
{_(
7476
'Note: in horizontal orientation, Y values are used for binning. If X values are provided, they are used as inputs to the histogram function which you can configure in the '
7577
)}
7678
<a onClick={() => setPanel('Style', 'Traces')}>{_('Traces')}</a>
77-
{_(' panel. If X values are omitted, the histogram function defaults to Count.')}
79+
{_(
80+
' under Style panel. If X values are omitted, the histogram function defaults to Count.'
81+
)}
7882
</HistogramInfoHorizontal>
7983
<Histogram2d>
8084
{_(
8185
'Note: X and Y Values are used for binning. If Z values are provided, they are used as inputs to the histogram function which you can configure in the '
8286
)}
8387
<a onClick={() => setPanel('Style', 'Traces')}>{_('Traces')}</a>
84-
{_(' panel. If Z values are omitted, the histogram function defaults to Count.')}
88+
{_(
89+
' under Style panel. If Z values are omitted, the histogram function defaults to Count.'
90+
)}
8591
</Histogram2d>
8692
<DataSelector label={_('I (Optional)')} attr="i" />
8793
<DataSelector label={_('J (Optional)')} attr="j" />

src/styles/components/sidebar/_main.scss

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
float: left;
1111
border-right: var(--border-default);
1212
flex-grow: 1;
13-
@include scrollbar();
13+
@include scrollbar(0px);
1414

1515
&__group {
1616
background-color: var(--sidebar-group-background-base);
@@ -102,8 +102,8 @@
102102
line-height: var(--font-size-medium);
103103
text-transform: capitalize;
104104
background-color: var(--sidebar-item-background-base);
105-
padding: var(--spacing-half-unit);
106-
padding-left: var(--spacing-base-unit);
105+
padding: 10px;
106+
padding-left: 16px;
107107
padding-right: var(--spacing-quarter-unit);
108108
text-align: left;
109109
border-bottom: var(--border-light);

0 commit comments

Comments
 (0)