Skip to content

Conversation

bruugey
Copy link
Collaborator

@bruugey bruugey commented Mar 25, 2020

✍️ Proposed changes

  • Add payment status badge to ProjectNav

🎟 Jira ticket: PD-547

🛠 Types of changes

  • Tooling (updates to workspace config or internal tooling)
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

✅ Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have run yarn changeset and documented my changes

Screen Shot 2020-03-25 at 10 57 33 AM

@bruugey bruugey requested review from hswolff and DesignerDave and removed request for hswolff March 25, 2020 14:59
@github-actions
Copy link
Contributor

github-actions bot commented Mar 25, 2020

Size Change: +9.79 kB (1%)

Total Size: 529 kB

Filename Size Change
packages/menu/dist/index.node.js 6.78 kB +1 B
packages/menu/dist/index.web.js 6.78 kB +2 B (0%)
packages/mongo-nav/dist/index.node.js 31.8 kB +4.89 kB (15%) ⚠️
packages/mongo-nav/dist/index.web.js 31.9 kB +4.89 kB (15%) ⚠️
ℹ️ View Unchanged
Filename Size Change
packages/badge/dist/index.node.js 1.92 kB 0 B
packages/badge/dist/index.web.js 1.92 kB 0 B
packages/box/dist/index.node.js 7.01 kB 0 B
packages/box/dist/index.web.js 7.05 kB 0 B
packages/button/dist/index.node.js 9.31 kB 0 B
packages/button/dist/index.web.js 9.35 kB 0 B
packages/card/dist/index.node.js 7.05 kB 0 B
packages/card/dist/index.web.js 7.08 kB 0 B
packages/checkbox/dist/index.node.js 74.1 kB 0 B
packages/checkbox/dist/index.web.js 74.1 kB 0 B
packages/code/dist/index.node.js 7.74 kB 0 B
packages/code/dist/index.web.js 7.79 kB 0 B
packages/emotion/dist/index.node.js 973 B 0 B
packages/emotion/dist/index.web.js 1.04 kB 0 B
packages/hooks/dist/index.node.js 2.72 kB 0 B
packages/hooks/dist/index.web.js 2.77 kB 0 B
packages/icon-button/dist/index.node.js 2.28 kB 0 B
packages/icon-button/dist/index.web.js 2.28 kB 0 B
packages/icon/dist/index.node.js 8.3 kB 0 B
packages/icon/dist/index.web.js 8.3 kB 0 B
packages/leafygreen-provider/dist/index.node.js 1.8 kB 0 B
packages/leafygreen-provider/dist/index.web.js 1.8 kB 0 B
packages/lib/dist/index.node.js 1.25 kB 0 B
packages/lib/dist/index.web.js 1.25 kB 0 B
packages/logo/dist/index.node.js 8.3 kB 0 B
packages/logo/dist/index.web.js 8.3 kB 0 B
packages/modal/dist/index.node.js 2.96 kB 0 B
packages/modal/dist/index.web.js 2.96 kB 0 B
packages/palette/dist/index.node.js 1.17 kB 0 B
packages/palette/dist/index.web.js 1.17 kB 0 B
packages/pipeline/dist/index.node.js 12.9 kB 0 B
packages/pipeline/dist/index.web.js 12.9 kB 0 B
packages/popover/dist/index.node.js 4.72 kB 0 B
packages/popover/dist/index.web.js 4.72 kB 0 B
packages/portal/dist/index.node.js 1.71 kB 0 B
packages/portal/dist/index.web.js 1.71 kB 0 B
packages/radio-box-group/dist/index.node.js 3.79 kB 0 B
packages/radio-box-group/dist/index.web.js 3.8 kB 0 B
packages/radio-group/dist/index.node.js 2.93 kB 0 B
packages/radio-group/dist/index.web.js 2.93 kB 0 B
packages/side-nav/dist/index.node.js 8.69 kB 0 B
packages/side-nav/dist/index.web.js 8.75 kB 0 B
packages/syntax/dist/index.node.js 24.1 kB 0 B
packages/syntax/dist/index.web.js 24.1 kB 0 B
packages/tabs/dist/index.node.js 9.51 kB 0 B
packages/tabs/dist/index.web.js 9.55 kB 0 B
packages/text-input/dist/index.node.js 3.19 kB 0 B
packages/text-input/dist/index.web.js 3.19 kB 0 B
packages/theme/dist/index.node.js 940 B 0 B
packages/theme/dist/index.web.js 940 B 0 B
packages/toggle/dist/index.node.js 4.29 kB 0 B
packages/toggle/dist/index.web.js 4.29 kB 0 B
packages/tooltip/dist/index.node.js 4.56 kB 0 B
packages/tooltip/dist/index.web.js 4.61 kB 0 B
packages/typography/dist/index.node.js 7.59 kB 0 B
packages/typography/dist/index.web.js 7.63 kB 0 B

compressed-size-action

@bruugey bruugey requested a review from hswolff March 26, 2020 13:20
Copy link
Collaborator

@DesignerDave DesignerDave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple comments!

margin-right: 20px;
`;

const Colors = {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It may be worth exporting this enum from badge!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good call honestly i'll do that now

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it already was 🤦‍♀, but using it now

z-index: 1;
${facepaint({
marginRight: ['20px', '14px', '20px'],
marginRight: ['16px', '14px', '16px'],
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wondering what this change is for?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To better align the two badges, as they are in the design spec

@bruugey bruugey requested a review from DesignerDave March 26, 2020 17:12
export { OrgNavLink, InteractionRingWrapper, OnPremUserMenu };
import ProjectStatusBadge from './ProjectStatusBadge';
import { RealmIcon, AtlasIcon, ChartsIcon } from './Icons';
export {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should switch to our export only interface!

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did we ever figure out what the blocking issue was that kept us from doing this in the first place?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TypeScript I believe. Newer version now supports it.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh I can't wait to do this everywhere then haha

@bruugey bruugey merged commit 64c03e7 into master Mar 26, 2020
@bruugey bruugey deleted the PD-547 branch March 26, 2020 21:08
bruugey added a commit that referenced this pull request Aug 9, 2023
* add project stauts badge

* changeset

* refactor

* use variant from badge

* set width to 32px manually on platform icons for submenu alignment

* changeset for menu

* change export patterns and fix platform icon bug
stephl3 pushed a commit that referenced this pull request Sep 29, 2025
* bump other packages

* major popover updates

* bump provider

* bump global provider

* bump cli

* rm popoverZIndex

* Create forty-days-rest.md

* Delete .eslintignore

* update lg to latest major

* Removes prop-types, and bump LG packages

* lint

* Update .eslintrc.js

* update storybook @lg-tools/storybook-utils

* lint
github-merge-queue bot pushed a commit that referenced this pull request Sep 30, 2025
…e to leafygreen-ui (#3165)

* LG-4333: VerticalStepper (#163)

* wip

* wip, transitions

* image

* break out step icon component

* tabindex

* line color

* fix story

* status to state

* padding

* style cleanup

* style updates

* consts

* add tests

* README

* changeset

* types and tests

* cleanup

* cleanup

* lint

* cleanup again

* some feedback

* update icon styles

* inert and tabindex

* use descedants

* remove extra div

* update package.json

* pass isCompleted to StepIcon

* cleanup

* feedback

* Version Packages (#169)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Fix naming of lg-private packages (#183)

* Fix lg-private packages naming

* Changeset

* LG-4395: feature walls major release (#186)

* LG-4395: vertical stepper design QA

* LG-4395: feature walls design QA

* Changeset

* Version Packages (#184)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Use actions slot prop for vertical stepper and activation steps (#191)

* VerticalStepper takes actions prop instead of primaryButtonProps and secondaryButtonProps

* ActivationSteps uses updated VerticalStepper API

* Changeset

* Address feedback

* Version Packages (#190)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* LG-4412: all sections render in card UI  (#194)

* Update Section styling and add optional renderInCard prop

* FeatureOverview renders in card, handles children container, and removes custom storybook decorator

* Templates always renders in card and export type Template

* UseCases always renders in card

* ActivationSteps type updates and remove storybook decorator

* Export types for ActivationStep, Template, and UseCase

* Changeset

* LG-4413: vertical stepper always renders description and media (#195)

* VerticalStepper always renders description and media

* Changeset

* Cleanup

* Version Packages (#197)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* LG-4670: feature walls component resizing (#294)

* Update vertical stepper button layout on smaller breakpoints

* Fix resizing in AccordionPanel

* Fix resizing in ExpandableGrid

* Feedback

* height is always available

* Version Packages (#292)

* Version Packages

* Version next @lg-private/vertical-stepper and update changelogs

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Stephen Lee <stephen.lee@mongodb.com>

* Updates Popover/Menu/Tooltip to TopLayer LG-4746 (#337)

* bump other packages

* major popover updates

* bump provider

* bump global provider

* bump cli

* rm popoverZIndex

* Create forty-days-rest.md

* Delete .eslintignore

* update lg to latest major

* Removes prop-types, and bump LG packages

* lint

* Update .eslintrc.js

* update storybook @lg-tools/storybook-utils

* lint

* Version Packages (#343)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* PNPM + Storybook 8 (private) (#426)

* pnpm import

* Delete yarn.lock

* workflows

* yarn -> pnpm docs

* workspace:^

* add missing global packages

* @storybook/test

* story ts fixes

* Adds missing peers

* rm lg.json

* bump base lg

* fix product icon builds

* fix sb

* bump peer deps

* rm  pnpm/action-setup@v4

* Revert "rm  pnpm/action-setup@v4"

This reverts commit 14198eb258e958ddaa0e480fee17cfddec5ce6fd.

* Update pnpm-lock.yaml

* apk

* apk all

* wget

* restore pnpm

* specific pnpm action

* workspace

* lint fix

* tools @ latest

* minor lint errors

* tools build dev deps

* typescript

* tsconfig

* package exports

* ts fixes

* Create rollup.config.mjs

* lint

* checkout main src

* ts fixes

* lint

* isolated async tests

* Update CloudNav.analytics.spec.tsx

* skip mongonav tests

* feature walls tests

* Create rollup.config.mjs

* lint fix

* Update pr.yml

* Update release.yml

* cache/restore

* chromatic + pnpm

* update cache restore paths

* cache save path

* addon 0.5.2-next.0

* addon 0.5.2-next.2

* chore(vertical-stepper): move from @lg-private to @leafygreen-ui scope

* chore(vertical-stepper): disable snapshot of LiveExample story

---------

Co-authored-by: Shaneeza <shaneeza.ali@mongodb.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Adam Thompson <2414030+TheSonOfThomp@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants