Problem/Motivation
The 'visual inline mode' when comparing two revisions of a node displays the default template instead of the full template.
e.g. node--default.html.twig instead of node--full.html.twig
The default view mode in Drupal is "full" even if full is not enabled.
See https://www.drupal.org/project/drupal/issues/2844203 for the Default/Full view modes issue.
Proposed resolution
The getViewModeOptionsByBundle() and related methods returns the wrong data.
As a workaround for D8, is to allow choosing the view modes.

| Comment | File | Size | Author |
|---|---|---|---|
| #10 | 3058111-add-default-view-mode-setting-3.patch | 3.69 KB | rishvi |
Issue fork diff-3058111
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
kirst25Comment #3
kirst25Comment #4
rishvi commentedWe also faced the inconvenience of not being able to set the default view mode on visual inline mode. It's not very clear to the site builders to choose a different view mode to be able to see the differences between revisions.
I would like to suggest a different approach to resolve this. Instead of replacing the default value with full, add a new config setting on visual inline to set the default view mode. Users can choose to set default or full as the default view mode on the visual inline layout.
Comment #5
rishvi commentedComment #6
jibranSounds like a great idea to me.
Should we add the 'diff' view mode as well?
Comment #7
rishvi commented@jibran Thanks for your response. I will upload a new patch with diff.
Thanks!
Comment #8
rishvi commentedHere's a new patch with diff view mode. Code is also updated to handle the case where a default view mode is not enabled on a bundle type.
Comment #9
rishvi commentedComment #10
rishvi commentedComment #11
rishvi commentedComment #12
anas_maw commentedPatch in 10 works for me, thanks
Comment #13
bkosborne+1 this is exactly what I'm looking for. But also the ability to completely hide this view mode selector as well.
Comment #14
idebr commentedPatch works as expected, thanks!
Comment #15
acbramley commentedPatch in #10 still applies to latest dev, I've triggered tests on D9.5 and D10. It would be great to finally get this committed!
Comment #19
acbramley commented