Skip to content
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

review - bottom navigation #127

Merged
merged 2 commits into from
May 22, 2023
Merged

review - bottom navigation #127

merged 2 commits into from
May 22, 2023

Conversation

hyesungoh
Copy link
Member

๐Ÿค” ํ•ด๊ฒฐํ•˜๋ ค๋Š” ๋ฌธ์ œ๊ฐ€ ๋ฌด์—‡์ธ๊ฐ€์š”?

close #125

๐ŸŽ‰ ๋ณ€๊ฒฝ ์‚ฌํ•ญ

  • review ํŽ˜์ด์ง€์—์„œ ์ž์ฃผ ์‚ฌ์šฉ๋˜๋Š” BottomNavigation ์ปดํฌ๋„ŒํŠธ๋ฅผ ๋งŒ๋“ค์—ˆ์–ด์š”
    • review intro stepย #116 ์˜ fixedBottomCss๋ฅผ ๋™์ผํ•˜๊ฒŒ ๊ฐ€์ ธ์™€ ์‚ฌ์šฉํ•˜๋ฉฐ ์˜ค๋ฒ„๋ผ์ด๋”ฉํ•ด ์‚ฌ์šฉํ•ด์š”

๐ŸŒ„ ์Šคํฌ๋ฆฐ์ƒท

แ„‰แ…ณแ„แ…ณแ„…แ…ตแ†ซแ„‰แ…ฃแ†บ 2023-05-22 แ„‹แ…ฉแ„Œแ…ฅแ†ซ 1 24 44

แ„‰แ…ณแ„แ…ณแ„…แ…ตแ†ซแ„‰แ…ฃแ†บ 2023-05-22 แ„‹แ…ฉแ„Œแ…ฅแ†ซ 1 24 51

@hyesungoh hyesungoh added the feat New feature or request label May 21, 2023
@hyesungoh hyesungoh self-assigned this May 21, 2023
@github-actions
Copy link

Bundle Sizes

Compared against 3cc1d7d

Route: No significant changes found

Dynamic import: None found.

@codecov-commenter
Copy link

Codecov Report

Patch and project coverage have no change.

Comparison is base (3cc1d7d) 95.90% compared to head (10a122d) 95.90%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #127   +/-   ##
=======================================
  Coverage   95.90%   95.90%           
=======================================
  Files          39       39           
  Lines         244      244           
  Branches       41       41           
=======================================
  Hits          234      234           
  Misses         10       10           

โ˜” View full report in Codecov by Sentry.
๐Ÿ“ข Do you have feedback about the report comment? Let us know in this issue.

export default meta;

export const Default = () => {
const [value, onChange, reset] = useInput();
Copy link
Collaborator

Choose a reason for hiding this comment

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

์˜ค useInput ํ›…์„ ์ด๋ ‡๊ฒŒ ์“ธ์ˆ˜์žˆ๊ตฐ์šฉ

Copy link
Collaborator

@oyeon-kwon oyeon-kwon left a comment

Choose a reason for hiding this comment

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

๊ณ ์ƒํ•˜์…จ์Šต๋‹ˆ๋‹ค!!

@hyesungoh hyesungoh merged commit 06eef33 into main May 22, 2023
@hyesungoh hyesungoh deleted the feat/review-bottom-nav branch May 22, 2023 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

review - bottom navigation
3 participants