Skip to content

Conversation

@OstafinL
Copy link
Contributor

@OstafinL OstafinL commented Aug 30, 2023

Question Answer
Tickets https://issues.ibexa.co/browse/IBX-3698
Bug fix? no
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

Added properties help_attr.with_icon|default(true) to display icons in the form_help block

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

{% block form_help -%}
{%- if help is not empty -%}
{%- set help_attr = help_attr|merge({class: (help_attr.class|default('') ~ ' ibexa-form-help')|trim}) -%}
{%- set with_icon = help_attr.with_icon is defined ? help_attr.with_icon: true -%}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{%- set with_icon = help_attr.with_icon is defined ? help_attr.with_icon: true -%}
{%- set with_icon = help_attr.with_icon is defined ? help_attr.with_icon : true -%}

?

Copy link
Contributor

@GrabowskiM GrabowskiM left a comment

Choose a reason for hiding this comment

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

with Darek fix

@OstafinL OstafinL changed the title IBX-3698: Contitional icon for form_help IBX-3698: Conditional icon for form_help Sep 1, 2023
@OstafinL OstafinL force-pushed the IBX-3698-conditional-icon-form-help branch from 74e3485 to 20e86af Compare September 6, 2023 06:32
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@dew326 dew326 merged commit d5af11c into 4.5 Sep 11, 2023
@dew326 dew326 deleted the IBX-3698-conditional-icon-form-help branch September 11, 2023 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.