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

IBX-2655: create/edit layout fixes #391

Merged
merged 3 commits into from
May 6, 2022

Conversation

lucasOsti
Copy link
Contributor

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

Fixed create/edit layout by removing boostrap class row and col-. All content should be aligned to the left sidebar

  1. 1376px width
    1376

  2. 1920px width
    1920

  3. 1920px width
    2560

Checklist:

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

@@ -1,3 +1,11 @@
.ibexa-edit-content {
@include edit-right-side-container;

&__fulfillment {
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure about this name

@lucasOsti lucasOsti force-pushed the IBX-2655-create-edit-layout-fixes branch from 8a58351 to 445549c Compare April 20, 2022 18:17
@lucasOsti lucasOsti requested review from GrabowskiM and dew326 April 20, 2022 18:18
@lucasOsti lucasOsti marked this pull request as draft April 22, 2022 09:58
@lucasOsti lucasOsti marked this pull request as ready for review April 22, 2022 13:13
@@ -4,6 +4,7 @@

{% set grouped_fields = grouped_fields|default([]) %}
{% set anchor_params = { items: grouped_fields|keys } %}

Copy link
Contributor

Choose a reason for hiding this comment

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

why newline here?

@@ -1,17 +1,22 @@
{% extends '@ibexadesign/ui/edit_section.html.twig' %}

{% set attr = { class: 'ibexa-content-type-edit__section' } %}
{% set one_column_layout = one_column_layout|default(false) %}
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 one_column_layout = one_column_layout|default(false) %}
{% set is_one_column_layout = is_one_column_layout|default(false) %}

@lucasOsti lucasOsti force-pushed the IBX-2655-create-edit-layout-fixes branch from 5810685 to fb8ac43 Compare April 29, 2022 08:18
@lucasOsti lucasOsti force-pushed the IBX-2655-create-edit-layout-fixes branch from c81d16f to d00832d Compare May 5, 2022 12:49
@lucasOsti lucasOsti requested review from tischsoic, dew326 and GrabowskiM and removed request for GrabowskiM May 5, 2022 12:52
@adamwojs adamwojs merged commit 19d3528 into ibexa:main May 6, 2022
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.

8 participants