etn-ccis / blui-react-component-library

Re-usable React components for use in Brightlayer UI applications
BSD 3-Clause "New" or "Revised" License
16 stars 10 forks source link

Remove divider from snippets & update content #855

Closed JeffGreiner-eaton closed 6 months ago

JeffGreiner-eaton commented 6 months ago

Fixes https://github.com/etn-ccis/blui-react-component-library/issues/775.

Changes proposed in this Pull Request:

Screenshots / Screen Recording (if applicable)

-

To Test:

Any specific feedback you are looking for?

-

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (a86f67f) 84.03% compared to head (a52c3eb) 84.03%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #855 +/- ## ======================================= Coverage 84.03% 84.03% ======================================= Files 58 58 Lines 1234 1234 Branches 158 158 ======================================= Hits 1037 1037 Misses 98 98 Partials 99 99 ``` | [Flag](https://app.codecov.io/gh/etn-ccis/blui-react-component-library/pull/855/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=etn-ccis) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/etn-ccis/blui-react-component-library/pull/855/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=etn-ccis) | `84.03% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=etn-ccis#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 6 months ago

Visit the preview URL for this PR (updated for commit a52c3eb):

https://blui-react-docs--pr855-fix-blui-5307-nav-gr-0vunn203.web.app

(expires Fri, 16 Feb 2024 20:32:56 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 34d39fa5aab0ea0cf95074e8e76f68829e7a8c65

JeffGreiner-eaton commented 6 months ago

I think the issue logged was about the wrong prop name. It should be titleDivider instead of divider.

We may need to replace that and not remove.

image so do we want to pass in the default to demonstrate?