Closed (fixed)
Project:
Custom Elements
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
12 Jun 2019 at 13:59 UTC
Updated:
2 Jul 2019 at 07:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
pagach commentedThe patch prevents usage of CE in admin pages.
Comment #3
fagoyes, I noticed this as well. However, I'm not sure whether we should check the admin routes vs. the admin themes.
I guess it makes sense that CE follows the theme used and since "Use the administration theme when editing or creating content" checkbox is applied. Is this the right check that Drupal uses to determine whether to switch to the admin theme?
Next, I think this patch should go into ce everywhere only - since otherwise CE is only enabled for specific view modes anyway. custom_elements_* view modes should always be rendered as such imo, as well in the admin theme.
Comment #4
pagach commentedI think isAdminRoute is correct way to check this since it returns true for editing/creating forms only if "Use the administration theme when editing or creating content" checkbox is checked.
Agreed that custom_elements_* view modes should always be rendered as such. Disabled CE only in ce everywhere.
Comment #5
fagothx, that makes. Patch looks and is working as advertised. Thus, merged it.