Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Follow up for #1951268: Convert /forum and /forum/% to new router, remove forum_forum_load(), forum_get_topics(), create Forum service
Problem/Motivation
ForumController adds css files individually, ideally this should use a library.
Proposed resolution
Add forum css files to a library and attach them in the controller as such.
Remaining tasks
Write the patch
Review
User interface changes
None
API changes
None
Comment | File | Size | Author |
---|---|---|---|
#3 | Screenshot 2013-10-01 07.09.37.png | 25.47 KB | larowlan |
#2 | drupal8.forum-module.2028113-2.patch | 1.53 KB | RoSk0 |
Comments
Comment #1
larowlanWhoops, tags cloned via dreditor
Comment #2
RoSk0Initial patch.
Comment #3
larowlanLooks fine to me.
Screenshot from manual testing - the forum icons (which are the main intent of that css file) are still present.
Comment #4
alexpottCommitted 69a8334 and pushed to 8.x. Thanks!