feat: fieldsets as tabs

[nabonham]

Original issue: #44

Here is my approach to this feature. Tried my best on doing this in a non-intrusive way. Use fieldsets_tabs to create the tabs, and set fieldset-tab fieldset-tab-<tabname> in Inline and ModelAdmin classes parameter, as in the example below:

class TestModelInline(TabularInline):
    model = InlineModel
    classes = ["fieldset-tab fieldset-tab-advanced"]

@admin.register(TestModel)
class TestModelAdmin(ModelAdmin):
    fieldsets_tabs = (("general", _("General")), ("advanced", _("Advanced Settings")))
    fieldsets = [
        (
            None,
            {
                "classes": ["fieldset-tab fieldset-tab-general"],
                "description": "A very short description",
                "fields": ["field1", "field2"],
            },
        ),
        (
            _("Named Fieldset"),
            {
                "classes": ["fieldset-tab fieldset-tab-advanced"],
                "fields": ["field3", "field4"],
            },
        ),
    ]
    inlines = [
        TestModelInline
    ]

[nabonham]

I forgot to compile the assets before commiting & PR.

[lukasvinclav]

First of all thank you very much for your PR. I was already thinking for some time how to do it and your code forced me to do it. Below you can check my PR and let me know what you think. In general, I used alpine.js and I tried to avoid adding new attributes for tab configuration.

#249

[nabonham]

First of all thank you very much for your PR. I was already thinking for some time how to do it and your code forced me to do it. Below you can check my PR and let me know what you think. In general, I used alpine.js and I tried to avoid adding new attributes for tab configuration.

#249

Hello Lukas, firstly, congratulations on the project. I hope I can continue contributing more frequently.

My approach was simplistic, and I based purely on the django-suit implementation that @minholi commented on in the issue, including using the jQuery instance available in django-admin.

I really liked your approach, especially the use of alpine.js. My only suggestion is to keep the functionality of the fieldset headers (when providing the name for them). In this case, if you want to not display the header, just pass None. In your implementation, if it has the tab class, it automatically discards the rendering of the header.