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

ui:(core, geo): fix input form width, change overly pane to fit-content #1395

Merged
merged 5 commits into from
Sep 21, 2023

Conversation

aziz-access
Copy link
Contributor

What is the current behavior? (You can also link to an open issue here)

this behavior related to #1360

What is the new behavior?

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications:

Other information:

@aziz-access aziz-access self-assigned this Sep 19, 2023
@aziz-access aziz-access added the bug Incorrect or unexpected behaviors // Anomalies de fonctionnement label Sep 19, 2023
@aziz-access aziz-access linked an issue Sep 19, 2023 that may be closed by this pull request
@aziz-access aziz-access added in progress In progress // Développement en cours to test Issues that need testing by PO // Demandes qui ont besoin d'être testées par un pilote labels Sep 19, 2023
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le package-lock ne devrait pas se retrouver dans cette PR

.cdk-overlay-pane:has(.mat-mdc-select-panel, .mat-mdc-autocomplete-panel) {
/* styles to apply if the content contains select-panel OR autocomplete-panel */
min-width: fit-content;
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour l'appliquer de manière globale, est-ce que le mat-select de Angular est brisé?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

il est brisé lorsque la largeur de matSelect est plus longue qu'une option, la largeur du panneau ne s'étend pas jusqu'à la largeur de matSelect

Copy link
Collaborator

@alecarn alecarn Sep 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça semble être un comportement voulu de Material, voir le dernier point. Pour ce "Hot Fix", est-ce qu'on pourrait l'appliquer localement seulement pour la composante problématique et on ré-analysera plus tard si on l'applique gloabalement.

@alecarn
Copy link
Collaborator

alecarn commented Sep 20, 2023

Comment que tu faisais pour le tester dans IGO2, quel contexte tu utilisais?

@aziz-access
Copy link
Contributor Author

Comment que tu faisais pour le tester dans IGO2, quel contexte tu utilisais?

-Filtre attributaire (Attribute filter)
image

.cdk-overlay-pane:has(.mat-mdc-select-panel, .mat-mdc-autocomplete-panel) {
/* styles to apply if the content contains select-panel OR autocomplete-panel */
min-width: fit-content;
}
Copy link
Collaborator

@alecarn alecarn Sep 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je crois que cela va tout de même appliquer le style globallement, tu pourrais le déplacer a l'intérieur du fichier ogc-filter-form-component.scss?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

j'ai essayé mais le style ne se charge pas

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ogc-filterable-form.component charger tout le ogc filtre, j'ai ajouter providers config pour résoudre l'issue:
MatSelect .cdk-overlay-pane et MatAutocomplete .cdk-overlay-pane:

providers: [
    {
      provide: MAT_SELECT_CONFIG,
      useValue: { overlayPanelClass: 'igo-overlay-panel-width' }
    },
    {
      provide: MAT_AUTOCOMPLETE_DEFAULT_OPTIONS,
      useValue: { overlayPanelClass: 'igo-overlay-panel-width' }
    }
  ]

@alecarn alecarn requested a review from pelord September 21, 2023 13:32
@alecarn
Copy link
Collaborator

alecarn commented Sep 21, 2023

@pelord prêt pour la revue

@pelord pelord merged commit 7056094 into release/16.0.0 Sep 21, 2023
1 check passed
@pelord pelord deleted the 1360-timefilter-style-regression branch September 21, 2023 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect or unexpected behaviors // Anomalies de fonctionnement in progress In progress // Développement en cours to test Issues that need testing by PO // Demandes qui ont besoin d'être testées par un pilote
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TimeFilter style regression
3 participants