Skip to content

Fix (UI) - Text area fields alignment #1176

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.

Sign up for GitHub

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

Open
RomainLvr wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix (UI) - Text area fields alignment #1176

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
14686d5
Fix (UI) - Text area fields alignment
RomainLvr Apr 2, 2026
60175ce
Fix fields bloc alignment for new items & User forms
RomainLvr Apr 2, 2026
5d65134
Follow the same logic as the Tag plugin
RomainLvr Apr 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 38 additions & 12 deletions templates/fields.html.twig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,16 +28,46 @@

{% import 'components/form/fields_macros.html.twig' as macros %}

{% set already_wrapped = item is instanceof('CommonITILObject') and container.fields['type'] == 'dom' %}
{% set dropdown_item = item is instanceof('CommonDropdown') and container.fields['type'] == 'dom' %}

{% if item is instanceof('Glpi\\Form\\Form') %}
{% set already_wrapped = true %}
{% set in_itilobject = item is instanceof('CommonITILObject') and container.fields['type'] == 'dom' %}
{% set in_form_builder = item is instanceof('Glpi\\Form\\Form') %}
{% set already_wrapped = in_itilobject or in_form_builder %}
{% set dropdown_item = item is instanceof('CommonDropdown') and container.fields['type'] == 'dom' %}

{% set textarea_display_options = {
'in_itilobject': {
'field_class': 'col-12',
'label_class': 'col-2',
'input_class': 'col-10',
'is_horizontal': true,
'align_label_right': true,
},
'in_form_builder': {
'field_class': 'col-12',
'label_class': 'form-label',
'input_class': 'w-100',
'is_horizontal': false,
'align_label_right': false,
},
'default': {
'field_class': 'col-12',
'label_class': 'col-xxl-2',
'input_class': 'col-xxl-10',
'is_horizontal': true,
'align_label_right': true,
},
} %}

{% if in_itilobject %}
{% set current_textarea_options = textarea_display_options.in_itilobject %}
{% elseif in_form_builder %}
{% set current_textarea_options = textarea_display_options.in_form_builder %}
{% else %}
{% set current_textarea_options = textarea_display_options.default %}
{% endif %}

{% if not already_wrapped and not dropdown_item%}

{% set class = item.isNewItem() or item is instanceof('User') ? 'col-xxl-12' : 'col-xxl-9' %}
{% set class = item.isNewItem() or item is instanceof('User') or container.fields['type'] == 'dom' ? 'col-xxl-12' : 'col-xxl-9' %}
<div class="col-12 {{ class }} flex-column">
<div class="d-flex flex-row flex-wrap flex-xl-nowrap">
<div class="row flex-row align-items-start flex-grow-1">
Expand Down Expand Up @@ -88,15 +118,11 @@
})) }}

{% elseif type == 'textarea' %}
{{ macros.textareaField(input_name, value, label, field_options) }}
{{ macros.textareaField(input_name, value, label, field_options|merge(current_textarea_options)) }}

{% elseif type == 'richtext' %}
{{ macros.textareaField(input_name, value, label, field_options|merge({
{{ macros.textareaField(input_name, value, label, field_options|merge(current_textarea_options)|merge({
'enable_richtext': true,
'field_class': 'col-12',
'label_class': '',
'input_class': '',
'align_label_right': false,
})) }}

{% elseif type == 'yesno' %}
Expand Down