On 01/23/2014 06:11 AM, vincent(a)massol.net wrote:
Hi devs,
I’m working to fix
http://jira.xwiki.org/browse/XWIKI-9910 but before I can fix it we
need to decide something since we have 2 possibilities.
- Option 1: The hidden flag is set at document translation level which means when the
user check the hidden flag it’s only for the current translation
- Option 2: The hidden flag is set at the default document level (not set at translated
doc level) which means there’s a single hidden flag
ATM the problem with XWIKI-9910 is that when the user checks the hidden flag, it’s set at
the translation level but when a translation is displayed the value shown is the one from
the default document.
Option 1 offers more use cases but:
- users may be surprised
- users need to be careful to edit the default doc if they wish to set the doc as hidden
for all translations
I’m not sure what option I prefer. Initially I was more for option 2 but I’m now
hesitating and leaning more towards option 1. Note that option 2 means one more DB upate
when saving a translated doc.
I'm not sure 2 is going to work that easily, since by default queries
don't filter by the "translation" flag. 2 means that we have to change
every query (impossible if we count user queries), or the way the search
APIs work (backwards incompatible).
So +1 for 1.
Use case: the master document is visible, and it is an important one
(legal contract, license, official documentation...). Translations are
being worked on. While a translation isn't approved, they'd like it to
be hidden.
UX proposal:
- when a translation is created, it copies the hidden field from the master
- when a user changes the master's hidden status, a dialog shows up
asking if all the translations should be changed as well or not
- when a user changes a translation's hidden status, a dialog shows up
asking for a confirmation if it's different from the master, warning
about the possible issues caused by a difference in the flag
- we display the hidden status of the translation in the UI
--
Sergiu Dumitriu
http://purl.org/net/sergiu