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

Popover non viene caricato all'interno del container impostato #352

Closed
tbiondo opened this issue May 2, 2024 · 4 comments
Closed

Popover non viene caricato all'interno del container impostato #352

tbiondo opened this issue May 2, 2024 · 4 comments

Comments

@tbiondo
Copy link

tbiondo commented May 2, 2024

Andrebbe aggiornato il namespace dell'attributo (bs) che viene assegnato all'elemento

@Input() set popoverContainer(container: 'body' | string | undefined) {

@AntoninoBonanno
Copy link
Collaborator

@tbiondo potresti descrivere meglio il problema, aggiungendo anche con un frammento di codice?

@tbiondo
Copy link
Author

tbiondo commented Jul 8, 2024

@AntoninoBonanno la funzione indicata inserisce l'attributo "data-container" in modo da poter impostare appunto il container al quale agganciare il popover, ma nelle ultime versioni di bootstrap l'attributo va impostato con il namespace "bs", infatti adesso non funziona.
Di conseguenza,
this.element.setAttribute('data-container', container); -> this.element.setAttribute('data-bs-container', container);
Ti apro una pull request se preferisci.

@AntoninoBonanno
Copy link
Collaborator

La documentazione ufficiale indica data-container ma su bootstrap è data-bs-container.

@astagi puoi controllare?

@tbiondo, @astagi ho aperto la PR #368. Potete effettuare dei test prima di unirla alla main? grazie

astagi pushed a commit that referenced this issue Jul 12, 2024
@astagi
Copy link
Member

astagi commented Jul 12, 2024

Mergiata

@astagi astagi closed this as completed Jul 12, 2024
github-actions bot pushed a commit that referenced this issue Jul 15, 2024
## [1.0.3](v1.0.2...v1.0.3) (2024-07-15)

### Bug Fixes

* **button:** added default button type ([ea19950](ea19950)), closes [#355](#355)
* multiple initialization ([39fad60](39fad60))
* **popover:** fixed popover container attribute ([6327e76](6327e76)), closes [#352](#352)
* top on sticky header and add method for close NavBar on mobile ([d1df044](d1df044))

### Features

* add regex for italian plate and set plate validators ([be076df](be076df))
* added output event ([f29bfd3](f29bfd3))
* **timeline:** added timeline component ([147b8f5](147b8f5))
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

No branches or pull requests

3 participants