-
Notifications
You must be signed in to change notification settings - Fork 51
ApplicationState Enhancements - Implementation
#2274
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 56 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
7b54b69
added new yielded media component
zamoore 481cb31
yielding new media component from parent ApplicationState component
zamoore 62a52d5
updating docs to better reflect component defaults
zamoore 23badf3
converted index from template-only component
zamoore aabc344
added default left alignment to ApplicationState index
zamoore db7bddb
updating application-state styles
zamoore 7f5aed7
moved the error code above the title and title icon
zamoore e52f3eb
updated text styles
zamoore 86e7837
alignment classes - wip
zamoore 2a524a8
fixed center alignment issue
zamoore 3a2c9fa
figured out an alternative that didn't require updating the structureβ¦
zamoore e08b983
adjusting footer padding
zamoore ebc298c
actions are displayed correctly
zamoore edb4446
cleaning up documentation
zamoore d595db8
adding visual test cases for larger images
zamoore c834b98
final using flexbox
zamoore 5696958
implemented left aligned version
zamoore 87c41f5
updated showcase page to include a βhighlightβ button
didoo 914f2d5
added `Button` as yielded component for the `ApplicationState::Footerβ¦
didoo b04179a
extended showcase with more use cases and reorganized existing content
didoo 0921439
added a changeset
zamoore 79abaa1
addressing broken color in icon
zamoore e06bd08
Update changeset
zamoore 99d0094
adding integration tests
zamoore b90192d
fixing align getter
zamoore f077cd2
self-pr review adjustments
zamoore 4e0501a
adding additional test for invalid align value
zamoore d0c3e76
adjusting spacing
zamoore 5a42f7c
removed old content from showcase for `ApplicationState`
didoo 2319640
Update changeset to follow team conventions
zamoore 8206c8d
reordering TS def for team conformity
zamoore 8a2a8bb
small update to the showcase
didoo 6cb232e
adjust title color
zamoore fbfc239
Update color for error code
zamoore 83d2eeb
Update icon color
zamoore 448f644
add use cases to the ApplicationState showcase where itβs inside a coβ¦
didoo be039bb
responging to pr feedback
zamoore c654e4f
updating tests to follow standard assertion patterns
zamoore 0a13e85
fix linting error
zamoore 6456c6d
added new integration test for the media component
zamoore bc7d905
remove footer border option
zamoore 7f14f83
removed test related to deleted code
zamoore 266094c
added components to export
zamoore 453ce24
added dropdown as potentially yielded footer item
zamoore f5fdaa6
added a bottom margin to the error code element
zamoore b7e835c
fixed color on error code
zamoore 8c40c99
added custom title tag option
zamoore f4d85d2
added showcase examples and tests for the titleTag attribute
zamoore 4a4ffa3
added new tests for contextual component rendering to index component
zamoore d4e7a05
added dropdown action to showcase
zamoore fc40402
responding to PR feedback; commenting on usage of named css colors inβ¦
zamoore f3610b3
updated the changeset to include comprehensive details on the scope oβ¦
zamoore 8b7dea4
fixed failing test
zamoore 6cd36ce
adding mroe examples in showcase
zamoore b581cb7
Update .changeset
zamoore 4cda0d6
addressing pr feedback
zamoore bee9d67
Apply suggestions from code review
zamoore 57046dd
`ApplicationState` Enhancements - Documentation (#2269)
majedelass File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| --- | ||
| "@hashicorp/design-system-components": minor | ||
| --- | ||
|
|
||
| `ApplicationState`: | ||
|
|
||
| - Spacing and alignment updates | ||
| - New `@align` (`left` (default), `center`) argument for aligning content | ||
| - Added new yielded `Media` child component | ||
|
|
||
| `ApplicationState::Header`: | ||
|
|
||
| - The header now supports an optional `@titleTag` argument that can override the default title element (`div`) | ||
|
|
||
| `ApplicationState::Footer`: | ||
|
|
||
| - The footer now yields `Button` and `Dropdown` components as well as `LinkStandalone` | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8 changes: 8 additions & 0 deletions
8
packages/components/src/components/hds/application-state/media.hbs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| {{! | ||
| Copyright (c) HashiCorp, Inc. | ||
| SPDX-License-Identifier: MPL-2.0 | ||
| }} | ||
|
|
||
| <div class="hds-application-state__media" ...attributes> | ||
| {{yield}} | ||
| </div> |
18 changes: 18 additions & 0 deletions
18
packages/components/src/components/hds/application-state/media.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| /** | ||
| * Copyright (c) HashiCorp, Inc. | ||
| * SPDX-License-Identifier: MPL-2.0 | ||
| */ | ||
|
|
||
| import TemplateOnlyComponent from '@ember/component/template-only'; | ||
|
|
||
| export interface HdsApplicationStateMediaSignature { | ||
| Blocks: { | ||
| default: []; | ||
| }; | ||
| Element: HTMLDivElement; | ||
| } | ||
|
|
||
| const HdsApplicationStateMediaComponent = | ||
| TemplateOnlyComponent<HdsApplicationStateMediaSignature>(); | ||
|
|
||
| export default HdsApplicationStateMediaComponent; |
5 changes: 5 additions & 0 deletions
5
packages/components/src/components/hds/application-state/types.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| export enum HdsApplicationStateAlignValues { | ||
| Left = 'left', | ||
| Center = 'center', | ||
| } | ||
| export type HdsApplicationStateAligns = `${HdsApplicationStateAlignValues}`; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.