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

[Feature Request] Please Add material tab control #1616

Closed
HamidRezaAshkiyan opened this issue Jan 23, 2020 · 14 comments
Closed

[Feature Request] Please Add material tab control #1616

HamidRezaAshkiyan opened this issue Jan 23, 2020 · 14 comments
Milestone

Comments

@HamidRezaAshkiyan
Copy link

No description provided.

@HamidRezaAshkiyan HamidRezaAshkiyan changed the title Feature Request: Please Add material tab control [Feature Request] Please Add material tab control Jan 23, 2020
@jespersh
Copy link
Contributor

In the meantime, see https://github.com/spiegelp/MaterialDesignExtensions/ for TabControl style

@HamidRezaAshkiyan
Copy link
Author

Very good thnaks.

But I think it is better to be on main library.

@ahmed-abdelrazek
Copy link
Contributor

the original maintainer was against it because he has this https://github.com/ButchersBoy/Dragablz

@Keboo
Copy link
Member

Keboo commented Jan 26, 2020

I have raised the question with the original author about including something in the main library. With the addition of additional target platforms it seem like that restriction may need to change. I have raised the concern and am leaving this issue open as a reminder.

@bebenins
Copy link
Contributor

Dragablz is sufficient for me, for example. The only thing I wish it included is control for shadow. The shadow itself is also not the same as MD library shadow.
Not sure what others are missing, this is just FYI I guess

@Jesus805
Copy link
Contributor

Would like to note that Dragablz has not been updated in 2 years. Also, ButchersBoy profile states that he has been taking a break from Open Source. Perhaps it would be worthwhile to implement Material Tab (For people who don't need need the drag out feature)

@lgreenlee
Copy link

I'd just like to throw a vote in for this - this is a pretty core piece of the Material Design "specification". Given this library provides nearly everything else it should include this as a style. I don't really want to add more dependencies to projects for just one piece of functionality. It even looks like the style was just commented out? Thanks for a great library it has been great to use!

@ahmed-abdelrazek
Copy link
Contributor

@Keboo is this coming with v4.4.0?

@MichelMichels
Copy link
Member

MichelMichels commented Dec 23, 2021

@Keboo is this coming with v4.4.0?

I don't think any work has been pushed for this issue. You could always send a PR yourself.

I stand corrected, see lower comments

@ahmed-abdelrazek
Copy link
Contributor

will see what i can do at the weekend

@Keboo
Copy link
Member

Keboo commented Dec 23, 2021

There is actually work that has been done on this. https://github.com/MaterialDesignInXAML/MaterialDesignInXamlToolkit/blob/master/MaterialDesignThemes.Wpf/Themes/MaterialDesignTheme.TabControl.xaml#L24-L202

It is however, just been left commented out. The reasons for this are historical and a bit convoluted. When I first started maintaining the project the author's guidance was to not include a TabControl style, and Dragablz was expected to fulfil that need. However, because Dragablz is not part of this same org on GitHub it has not received the same level of updates and has fallen behind with notable issues being no theme brush updates for latest MDIX, and no netcore3.1+ support. This is leaving a hole in the library which is where this feature request is coming from.

I think it is probably reasonable to go ahead and uncomment that code (and validate that it is still correct).

@ahmed-abdelrazek
Copy link
Contributor

yep that was my plan

@MichelMichels
Copy link
Member

@ahmed-abdelrazek and I submitted PR requests to resolve this issue.

@Keboo Keboo added this to the 4.4.0 milestone Dec 29, 2021
@Keboo
Copy link
Member

Keboo commented Dec 29, 2021

Re-opening for now, just until the PR merges into master

@Keboo Keboo reopened this Dec 29, 2021
@Keboo Keboo closed this as completed Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants