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

(#425)(#427) Update styles for copenhagen #433

Merged
merged 3 commits into from
Dec 5, 2024
Merged

Conversation

st3phhays
Copy link
Member

@st3phhays st3phhays commented Dec 5, 2024

Description Of Changes

This commit addresses two style related items. For nested lists, they
will now be displayed as they should, instead of one long list regardless
of their nested status. In addition, styles have been added to allow
the usage and display of the Tab component. This has been structured in
such a way, that the example from the Bootstrap docs can be copied and
used directly. See example directly at:
https://getbootstrap.com/docs/5.3/components/navs-tabs/#javascript-behavior

Motivation and Context

These needed to be fixed up to work correctly.

Testing

  1. Follow instructions here: (#31)(#32) Nested lists and Tab component copenhagen_theme#33

Change Types Made

  • Bug fix (non-breaking change).
  • Feature / Enhancement (non-breaking change).
  • Breaking change (fix or feature that could cause existing functionality to change).
  • Documentation changes.
  • PowerShell code changes.

Change Checklist

  • Requires a change to the documentation.
  • Documentation has been updated.
  • Tests to cover my changes, have been added.
  • All new and existing tests passed?
  • PowerShell code changes: PowerShell v3 compatibility checked?

Related Issue

#425
#427

@st3phhays st3phhays self-assigned this Dec 5, 2024
This fixes up the nested list styles for the copenhagen theme. Previously
these styles were not allowing nested lists, and was making everything,
even nested lists, look like one big list with no indentations. Now,
everything appears as it should which allows content to be more readable.
This adds styles for the Tab component to the copenhagen theme css.
On most repositories, classes are used on the component to define these
styles, but, to ease development on this specific repository the styles
have been added directly to the css. This is so that examples can be
copied directly from the Bootstrap docs, and still be styled up to
choco-theme.
Copy link
Contributor

@alexaveldanez alexaveldanez left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@alexaveldanez alexaveldanez merged commit 94c3f12 into main Dec 5, 2024
3 checks passed
@st3phhays st3phhays deleted the release/0.8.3 branch December 12, 2024 16:06
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.

2 participants