Apply suggestions from code review

Co-authored-by: Caroline Horn <[email protected]>

Author: rshen91

src/plugins/shared_ux/public/components/toolbar/buttons/icon_button_group/icon_button_group.mdx

@@ -1,20 +1,20 @@
 ---
-id: sharedUX/Components/SolutionToolbar/Quick_Group
-slug: /shared-ux/components/toolbar/solution_toolbar/quick_group
-title: Solution Toolbar Quick Group
-summary: 'An array of buttons for the solution toolbar'
+id: sharedUX/Components/Toolbar/Icon_Button_Group
+slug: /shared-ux/components/toolbar/icon_button_group
+title: Toolbar Icon Button Group
+summary: 'An array of icon-only buttons for use in a toolbar'
 tags: ['shared-ux', 'component']
 date: 2022-02-23
 ---
 
 > This documentation is in-progress.
 
-This component requires a prop that consists of a series of buttons that can then be displayed based on the number of buttons desired. An example of a button that can be part of an array of quick buttons is included below:
+This component requires a prop that consists of a series of buttons that can then be displayed based on the number of buttons desired. An example of a button that can be part of an array of icon buttons is included below:
 
 ```
   {
     label: 'Text',
     onClick: clickHandler,
-    IconType: 'visText',
+    iconType: 'visText',
   }
 ```

src/plugins/shared_ux/public/components/toolbar/buttons/icon_button_group/icon_button_group.stories.tsx

@@ -13,8 +13,8 @@ import { IconButtonGroup } from './icon_button_group';
 import mdx from './icon_button_group.mdx';
 
 export default {
-  title: 'Toolbar/Toolbar Icon Button Group',
-  description: 'A collection of buttons that is a part of the solution toolbar.',
+  title: 'Toolbar/Icon Button Group',
+  description: 'A collection of buttons that is a part of a toolbar.',
   parameters: {
     docs: {
       page: mdx,
@@ -46,7 +46,6 @@ const quickButtons = [
   },
   {
     label: 'Markup',
-    getLegend: '',
     onClick: action('onMarkupClick'),
     iconType: 'visVega',
   },
@@ -62,7 +61,7 @@ export const ConnectedComponent: Story<{ quickButtonCount: number }> = ({ quickB
 };
 
 ConnectedComponent.argTypes = {
-  quickButtonCount: {
+  buttonCount: {
     defaultValue: 2,
     control: {
       type: 'number',

src/plugins/shared_ux/public/components/toolbar/buttons/icon_button_group/icon_button_group.tsx

@@ -16,7 +16,7 @@ import {
   EuiButtonGroupProps,
 } from '@elastic/eui';
 import { IconButtonGroupStyles } from './icon_button_group.styles';
-export interface QuickButton {
+export interface IconButton {
   /**
    * The accessible button label
    */
@@ -40,16 +40,16 @@ export interface Props {
   /**
    * Array of `QuickButton`s
    */
-  buttons: QuickButton[];
+  buttons: IconButton[];
 }
 
-type Option = EuiButtonGroupOptionProps & Omit<QuickButton, 'label'>;
+type Option = EuiButtonGroupOptionProps & Omit<IconButton, 'label'>;
 
 export const IconButtonGroup = ({ buttons, legend }: Props) => {
   const euiTheme = useEuiTheme();
   const iconButtonGroupStyles = IconButtonGroupStyles(euiTheme);
 
-  const buttonGroupOptions: Option[] = buttons.map((button: QuickButton, index) => {
+  const buttonGroupOptions: Option[] = buttons.map((button: IconButton, index) => {
     const { label, title = label, ...rest } = button;
 
     return {

src/plugins/shared_ux/public/components/toolbar/buttons/primary/primary.mdx

@@ -1,5 +1,5 @@
 ---
-id: sharedUX/Components/ToolbarButton
+id: sharedUX/Components/Toolbar/Primary_Button
 slug: /shared-ux/components/toolbar/buttons/primary
 title: Toolbar Button
 summary: An opinionated implementation of the toolbar extracted to just the button.
@@ -9,4 +9,4 @@ date: 2022-02-17
 
 > This documentation is in-progress.
 
-This button is a part of the toolbar component. This button has primary styling and requires a label. OnClick handlers and icon types are supported as an extension of EuiButtonProps. Icons are always on the left of any labels within the button.
+This button is a part of the toolbar component. This button has primary styling and requires a `label`. Interaction (`onClick`) handlers and `iconType`s are supported as an extension of EuiButtonProps. Icons are always on the left of any labels within the button.

src/plugins/shared_ux/public/components/toolbar/buttons/primary/primary.stories.tsx

@@ -12,8 +12,8 @@ import { ToolbarButton } from './primary';
 import mdx from './primary.mdx';
 
 export default {
-  title: 'Toolbar/Toolbar Button',
-  description: 'A primary button that is a part of toolbar.',
+  title: 'Toolbar/Primary button',
+  description: 'A primary button that is a part of a toolbar.',
   parameters: {
     docs: {
       page: mdx,