Closed (fixed)
Project:
Olivero
Version:
8.x-1.x-dev
Component:
Meetings
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
24 Aug 2020 at 14:41 UTC
Updated:
22 Sep 2020 at 14:19 UTC
Jump to comment: Most recent
Time: 2pm UTC (10am Eastern)
Meeting Location: #d9-theme channel on drupal.slack.com
| Kat Shaw | Hello |
| proeung | Good Morning/Afternoon! |
| komal k | Hello |
| hansa11 | Hello everyone :wave: |
| shaal | Hello |
| kiran.kadam911 | Hello Everyone! |
| mherchel | here! |
| brianperry | Hey there. |
| KarinG | Hello! :wave: |
| sd9121 | Hello |
| proeung | @mherchel and I are looking to submit a new patch to Core for review this week. :fingerscrossed: |
| mherchel | @Kat Shaw , @mgifford (he/him) and I had a call with the National Federation of the Blind last week! We set up a site for them to do testing on, and expect the testing results (and videos) within two weeks |
| mherchel | super excited about that |
| mherchel | Other than that, I haven’t had time to put in much work into the theme last week |
| proeung | @mherchel Correct me if I'm wrong, but I believe we addressed the majority of the feedback that are in this comment (#3111409: Add new Olivero frontend theme to Drupal 9.1 core as beta). We should be ready to resubmit a new patch up to Core soon? |
| mherchel | I think so… |
| mherchel | need to take another look :slightly_smiling_face: |
| proeung | Great! Really excited to get a new patch submitted for Core committer's review. |
| proeung | This thread is the place to bring up specific issues that need some extra discussion. |
| mherchel | We had a number of people test the following issue, but were unable to reproduce. If anyone runs into this, let us know #3168091: Unable to add new sections in Layout Builder |
| KarinG | ds -> are they using display suite as well? |
| mherchel | Potentially.. I’d love to see if that’s an issue. We did override some layout builder templates, but honestly, I think most of it was just markup / class changes |
| KarinG | Yes they are using display suite - "Display Suite classes breaks Layout Builder if Display Suite is not installed" - not sure how that's a theme issue |
| KarinG | I ran into this -> #3166156: Fix for Error: [] operator not supported for strings in olivero_preprocess_form_element_label() - Fix for Error: [] operator not supported for strings in olivero_preprocess_form_element_label() -> turned out to be a smart_date issue. Fixed now, but it's so easy to think it's a theme issue. |
| mherchel | Nice! |
| mherchel | yeah, looking into the layouts/DS issue more… I think we’re only extending CSS. Not even overriding the templates. (edited) |
| proeung | Feel free to add other suggestions here. |
| mherchel | I need more PM type help. Something that took a lot of time was breaking down the core comment review into individual issues and then responding to each of them. I’d love if someone wanted to take on that role. |
| proeung | We have several tickets that need additional work. Please help process some of these issues. https://www.drupal.org/project/issues/olivero?status=13 |
Participants:
Kat Shaw, proeung, komal k, hansa11, shaal, kiran.kadam911, mherchel, brianperry, KarinG, sd9121
Comments
Comment #2
katannshaw commentedAttending
Comment #3
hansa11 commentedAttending!
Comment #4
shaalAttending
Comment #5
karingIci!
Comment #6
brianperryAttending.
Comment #7
proeungAttended
Comment #8
sd9121 commentedAttended
Comment #9
komalk commentedAttending!
Comment #11
mtift