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.
in the function entity_translation_admin_paths you are overwriting the normal translation module's admin path settings however you are not using the check if (variable_get('node_admin_theme')) { so it forces the translation page onto the admin theme always even if you have it set to use the front theme for content editing.
Comment | File | Size | Author |
---|---|---|---|
#5 | admin-theme-fix-1377542-5.patch | 1.59 KB | bforchhammer |
Comments
Comment #1
Daemon_Byte CreditAttribution: Daemon_Byte commentedAlso in the entity info alter array you are hard coding the admin theme there also in $entity_info[$entity_type]['translation']['entity_translation']['theme arguments']
Comment #2
timofey CreditAttribution: timofey commented+1 Subscribing
can we just change line 101 of entity_translation.module
from
'theme arguments' => array('admin_theme'),
to
'theme arguments' => array('default_theme'),
Comment #3
AdamGerthel CreditAttribution: AdamGerthel commentedI've stumpled upon this as well. Paths such as "node/X/translate" and "node/X/translate/add/X/X" shouldn't be using the administration theme when the administration is explicitly set to not be active when editing content. It also seems as if other theme manipulation modules such as Theme Key are unable to override the theme on those paths.
Comment #4
bforchhammer CreditAttribution: bforchhammer commentedMarked #1770272: Respect admin theme setting: "use the administration theme when editing or creating content" as duplicate. Note that that issue contains a mini-patch for the problem of the OP, but it looks like we need a new one which also removes the 'theme arguments' key!?
Comment #5
bforchhammer CreditAttribution: bforchhammer commentedHere's a new patch which...
Comment #6
Erik.Johansson CreditAttribution: Erik.Johansson commentedGot the same problem and the patch doesn't fix my problem.. The entity translation page is stuck using admin theme even if we are using theme key. Any updates on this?
Comment #7
bforchhammer CreditAttribution: bforchhammer commentedYou may have to clear your (menu) cache once...
Comment #8
plachCommitted and pushed, thanks!