Skip to content

Feat/tooltips#2779

Open
samialfattani wants to merge 6 commits into
pallets-eco:masterfrom
samialfattani:feat/tooltips
Open

Feat/tooltips#2779
samialfattani wants to merge 6 commits into
pallets-eco:masterfrom
samialfattani:feat/tooltips

Conversation

@samialfattani
Copy link
Copy Markdown
Contributor

This PR adds the feature of enabling the Bootstrap tooltip where it can be inserted within adding a new view like this:

    view = MyModelView(Model1, param, name="Model1", tooltip="tooltip of Model1")
    admin.add_view(view)
image

Moreover, users can add tooltip anywhere using normal Bootstrap tooltip as follows:

<a title="tooltip message with <b>HTML enabled</b>"
     href="/admin/" 
     data-role="tooltip"
     data-toggle="tooltip" data-placement="bottom" data-html="true"
>
/admin/
</a>

Tests and Docs are updated

@samialfattani samialfattani marked this pull request as ready for review February 2, 2026 10:38
@stajilov
Copy link
Copy Markdown

stajilov commented Feb 5, 2026

Not bad, looks useful

@samialfattani
Copy link
Copy Markdown
Contributor Author

no body reviewed it yet 🙄

@ElLorans
Copy link
Copy Markdown
Contributor

Sorry for the late reply, maybe it's me but I do not see a big advantage. This might also bring more chaos while we have so many PRs for a new theme.

@samialfattani
Copy link
Copy Markdown
Contributor Author

Sorry for the late reply, maybe it's me but I do not see a big advantage. This might also bring more chaos while we have so many PRs for a new theme.

i don't think so, how could it lead to chaose where the html changes made on the current theme only. if some other themes are going to be added then they can present the tooltip in thier own way. Anyway if you intend to close this PR pleaes let me know so i won't put more effort here nor resolving the conflict.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants