refactor(nav-drawer): update themes

IgniteUI / igniteui-webcomponents

Ignite UI for Web Components is a complete library of UI components, giving you the ability to build modern web applications using encapsulation and the concept of reusable components in a dependency-free approach.

https://www.infragistics.com/products/ignite-ui-web-components

Other

113 stars 2 forks source link

refactor(nav-drawer): update themes #1167

Closed desig9stein closed 1 month ago

desig9stein commented 2 months ago

closes #1152

didimmova commented 2 months ago

Comments are still not resolved.

didimmova commented 2 months ago

In Light Fluent theme, the icon of the item should be primary-500, when idle, now it's primary-700

didimmova commented 2 months ago

Are we going to implement the sizes of the mini navdrawer? I suggest putting a min-width on it that is the size of the smallest one (fluent) and it will grow according to the padding and icon size.

didimmova commented 2 months ago

The border of the navdrawer is all around only in fluent. All other themes have a border only on the inline-end side.

imincheva commented 1 month ago

Bootstrap:

Border should be inside. The total width of the mini should be 88px.
The total width of the default one should be 240px (including the border).
Background color on hover should be grays.200 80% in dark mode.

AnjiManova commented 1 month ago

Material

The border should be inside. The total width of the mini should be 56px. The total width of the default one should be 240px (including the border).

sbayreva commented 1 month ago

Fluent

For both light and dark modes, default open and mini variants the border of the whole component should be positioned inside. Now, it is outside and the width of the whole component grows with a couple of pixels. For the open default nav drawer the width should be 240px (+ border) and 40px (+ border) width for the mini variant .

imincheva commented 1 month ago

Indigo

The nav drawer should have 8px padding on all sides