Skip to content

AB#128960 grid styles enhancement#2849

Merged
AntoineRelief merged 5 commits intonextfrom
AB#128960-grid-styles-enhancement
Apr 22, 2026
Merged

AB#128960 grid styles enhancement#2849
AntoineRelief merged 5 commits intonextfrom
AB#128960-grid-styles-enhancement

Conversation

@Maximo-reliefapps
Copy link
Copy Markdown
Contributor

@Maximo-reliefapps Maximo-reliefapps commented Apr 9, 2026

Description

Two changes have been made in the rule styles of a layout for a grid.

  1. There is a new field, "Display as badge". When this field is marked as true, the style will be applied it the form of a badge instead of to the whole cell.
    This field will be active by default on newly created style rules.

  2. Newly created styles rules will have one of the "apply styles field" selected by default, the first one. This has been implemented with the objective of having "Apply style to" option be by default "Only selected columns".

Useful links

Type of change

  • New feature (non-breaking change which adds functionality)
  • Improvement (refactor or addition to existing functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

  • Selec the display as badge option and check that it correctly styles the cells.
  • Test a cell with overflow with the display as badge option.
  • Created a new style rule and made sure the display as badge & only selected columns options where selected.

Screenshots

Screenshot 2026-04-09 at 09 24 54 Screenshot 2026-04-09 at 09 46 09 Screenshot 2026-04-09 at 09 46 26

Checklist:

( * == Mandatory )

  • * I have set myself as assignee of the pull request
  • * My code follows the style guidelines of this project
  • * Linting does not generate new warnings
  • * I have performed a self-review of my own code
  • * I have put the ticket for review, adding the oort-frontend team to the list of reviewers
  • * I have commented my code, particularly in hard-to-understand areas
  • * I have put JSDoc comment in all required places
  • * My changes generate no new warnings
  • * I have included screenshots describing my changes if relevant
  • * I have selected labels in the Pull Request, according to the changes with code brings
  • I have made corresponding changes to the documentation ( if required )
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

More explanation

https://www.loom.com/share/05a716d61b9744faaf51fb304c21d1e5?sid=f87cf896-582a-4f76-93ae-8ceed801b145

@Maximo-reliefapps Maximo-reliefapps self-assigned this Apr 9, 2026
@Maximo-reliefapps Maximo-reliefapps added enhancement New feature or request changes Changes made to existing features labels Apr 9, 2026
ngOnInit(): void {
this.fieldNodes = this.setFieldNodes();

console.log(this.form);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Maximo-reliefapps
Don't forget to remove useless console logs before review 👍

@AntoineRelief AntoineRelief merged commit 24fed47 into next Apr 22, 2026
1 check passed
@AntoineRelief AntoineRelief deleted the AB#128960-grid-styles-enhancement branch April 22, 2026 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changes Changes made to existing features enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants