kirby icon indicating copy to clipboard operation
kirby copied to clipboard

Existing values for `translate: false` - keep or delete?

Open distantnative opened this issue 4 years ago • 5 comments

There seem to be conflicting views on what should happen when values in non-default language content files exist for fields with translate: false:

  • https://github.com/getkirby/kirby/issues/1859
  • https://github.com/getkirby/kirby/issues/2337

Moreover, it seems that we are not very coherent at the moment: Some actions keep the values around, others remove them.

Opening this issue to tie the discussions together.

distantnative avatar Dec 15 '19 14:12 distantnative

Vote delete :)

Any secondary language content stored in a field that is translate: false is invalid by definition. The panel should not accept invalid content and consequently clear it.

moritzebeling avatar Dec 16 '19 10:12 moritzebeling

I'm for deletion as well but like to point out again that the UI should notify the user about the issue:

The panel knows about this without any user interaction. So I'd expect two things in the panel:

  1. Either there should be an error message stating that there is a translation for an untranslatable field offering a way to dismiss this translation.
  2. Or the panel should just display the correct version from the default language and display the unsaved changes bar so that the translation is erased after hitting save.

See https://github.com/getkirby/kirby/issues/1859#issuecomment-542096208

nilshoerrmann avatar Dec 16 '19 10:12 nilshoerrmann

Agree, the user would have to manually save that edit. I think, a little red modal could inform the user that content is invalid (same as when changing templates) and that saving the page would solve that. But still, the user could refuse and go back without saving.

moritzebeling avatar Dec 16 '19 11:12 moritzebeling

I'm also pro deletion. I wonder why @tillprochaska want's to keep content thats not editable in the panel around as described in #2337. For me it only happens as an error that I forget to set a field to translatable: false and then have to dig into the txt to remove the value (the same thing that happens to Nils and Bruno).

mrflix avatar Nov 13 '21 13:11 mrflix

This issue has been automatically marked as stale because it has not had recent activity. This is for us to prioritize issues that are still relevant to our community. It will be closed if no further activity occurs within the next 14 days. If this issue is still relevant to you, please leave a comment.

stale[bot] avatar Aug 10 '22 10:08 stale[bot]