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

Add alt section workflow #394

Closed
wants to merge 28 commits into from
Closed

Conversation

Maxxen
Copy link
Contributor

@Maxxen Maxxen commented Mar 6, 2020

Description of the Change

This is a major rework of how the front end blocks are structured. This PR primarily introduces a hierarchical block based workflow, but also reconfigures how some of the current block templates behave in order to fit into the hierarchy and to properly make use of materialize.css. Since this was such a big change I also took the liberty to implement some basic missing blocktypes based on materialize components.

This draft PR is to track progress for the finishing touches and to gather feedback and input.

In short blocks are now grouped into the following hierarchy:
Level 0: Sections

  • Section
  • Divider

Level 1: Layout

  • Columns
  • Flex Columns
  • Logos
  • Counters
  • Contact Cards

Level 2: Standard

  • Collapsible container

Level 3: Basics

  • Heading
  • Paragraph
  • Responsive image
  • Image overlay
  • Icon
  • Divider
  • Table
  • Contact Card
  • Map

Changelog

  • Added table block

  • Added divider block

  • Added icon block

  • Added collapsible container block

  • Added container block

  • Added button block

  • Added button-group block

  • Added paragraph block with selectable text-justification

  • Merged Map block

  • Added section block with "padding (small | medium | large)" and "extend full width" properties

  • Reworked Home and Web -pages to include body field blocks with section html tags

  • Reworked ResponsiveImage block to be properly responsive

  • Reworked Two column grid -> Flex Columns.

  • Reworked image overlay to include configurable height property, button group.

  • Reworked contact card template to now scale to parent width.

  • Removed image-description block (because it is just weird and redundant)

  • Removed image-icons block (same as above)

Image-description block and image-icons is removed because it just doesn't work well with the new layout system and it can practically be implemented with the new blocks. counters block should probably also be removed for the same reason but stands as is for now because the "darken" functionality is kinda unique and standardizing themeing is a pandoras box not to be opened now #329 .

Future improvements:

One day I would like to create custom forms for some of the blocks to make editing in the admin interface slightly smoother. Specifically Table block, Map block and maybe even Section (remove indentation). Im thinking adding a "horizontal list" form may be a good first step, also making all blocks collapsible would be nice.

@Maxxen Maxxen mentioned this pull request Mar 7, 2020
@Maxxen Maxxen mentioned this pull request Mar 10, 2020
@Maxxen Maxxen marked this pull request as ready for review March 10, 2020 22:14
@Maxxen Maxxen requested a review from dakaza98 March 10, 2020 22:14
@Maxxen
Copy link
Contributor Author

Maxxen commented Mar 10, 2020

Now everything should be in order. I can't really think of anything else I would like to add for Forskå's sake in the near future. There will probably be some fixes and tweaks todo, but I think it is probably best to try to push this out before this branch grows even more out of control.

@dakaza98
Copy link
Member

This is a nice feature that I think we definitely should have. There is an issue however with webpages that currently exist. If we you apply this PR and have pages that has been created the old way, they get broken and the previous content does not get shown. You also can't access the previous content when editing them. We need to find some way to solve this, perhaps we can make a script that converts all the sites to the new format?

When looking at the page before:
om-karen-before

After:
om-karen-after

When editing the page before:
om-karen-edit-before

After:
om-karen-edit-after

@Maxxen Maxxen mentioned this pull request Apr 4, 2020
@Maxxen
Copy link
Contributor Author

Maxxen commented Apr 4, 2020

Reworked in #399

@Maxxen Maxxen closed this Apr 4, 2020
@dakaza98 dakaza98 deleted the add-alt-section-workflow branch May 25, 2020 15:33
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.

None yet

2 participants