Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Comment | File | Size | Author |
---|---|---|---|
#3 | multilingual-demo--beta13--2537456-3.patch | 15.7 KB | loopduplicate |
#2 | multilingual-demo--beta13--2537456-2.patch | 15.82 KB | loopduplicate |
Comments
Comment #1
loopduplicateComment #2
loopduplicateAlthough beta13 isn't out yet, I decided to see if the latest from core worked with this profile. I found that the markup for titles in views was improperly escaped. When I inspected the markup, I knew what to do :) I updated the view to change the markup of the title by removing the rewrite rule. The rule included an inline css style that was being stripped anyway. The rule also made it so an h3 element contained a div, which isn't really the best. I think that is why core now escapes this rewrite. See my notes below for details.
I'm attaching a patch that shouldn't be committed. It has the make file updated to test this with the latest from core. When beta13 comes out, the patch can be updated so the make file uses beta13. I re-exported the view config file instead of manually updating it. That's why it's so different. My notes below should clarify what I did.
Comment #3
loopduplicateHere's the new patch for when beta13 comes out, which I think might be later today.
Comment #4
loopduplicatemarking this postponed until beta13 comes out
Comment #5
Gábor HojtsyIts out now!
Comment #6
loopduplicateSimplytest is having troubles now so I didn't test there but I did test locally and everything looks good :) I think this is ready to commit.
Comment #8
Gábor HojtsyI could not verify the patch either on simplytest.me and did not have time to test locally but I trust your judgement. I removed the uuid from the view on commit. Why were all those updates needed to that view? Eg. the altered title view?
Comment #9
Gábor HojtsyStill does not work after I committed, so opened #2542882: Multilingual demo dependencies problem.
Comment #10
loopduplicateThe profile works on simplytest.me now. See #2542882
Comment #11
Gábor HojtsyYay, thanks a lot!
Comment #12
Kristen PolIt's working fine on simplytest.me now. Oh, @loopduplicate already said that in #10. :)
Comment #13
Gábor HojtsyJust opened #2546220: Update to Drupal 8 beta 14, that is also out now :)