Display heading level & Expand heading level
Expected behaviour
Display heading level & Expand heading level book setting fields should support all values in our PMCBook Domain attribute fields so that we don't break rendering for legacy content we are migrating to the new CMS.
Based on current domain tool data, we should at least support
Display heading level |
---|
99 |
1 |
2 |
3 |
5 |
1:nojava |
2:nojava |
3:nojava |
<blank> |
Expand heading level |
---|
1 |
2 |
<blank> |
Current behaviour
Display heading level - only allows digits (up to "4") & is a required field that does not allow blanks
Expand heading level - is a required field that does not allow blanks
Steps to reproduce
- create any book
- try to edit settings for Display heading level & Expand heading level
Agreed solution
Agreed solution for display heading level
- Change display heading level field in Books, Book Org templates, and Collection templates, to a dropdown list instead of a numbered carousel.
- Remove validation in the field so it is allowed the value of being blank or empty
- In the dropdown list, support the following string values:
Display heading level |
---|
1 |
2 |
3 |
4 |
5 |
99 |
1:nojava |
2:nojava |
3:nojava |
<blank> |
Note when the user selects <blank>
in the dropdown, the value should be saved as empty and therefore sent to NCBI empty, not as the string <blank>
.
Note that the domain default appears to be 4, so that can remain the default value, which the user can change via the dropdown.
Agreed solution for expand heading level
- Change expand heading level field in Books, Book Org templates, and Collection templates, to a dropdown list instead of a numbered carousel.
- Remove validation in the field so it is allowed the value of being blank or empty
- In the dropdown list, support the following string values:
Expand heading level |
---|
1 |
2 |
3 |
4 |
<blank> |
Note when the user selects <blank>
in the dropdown, the value should be saved as empty and therefore sent to NCBI empty, not as the string <blank>
.
Note that the domain default appears to be 4, so that can remain the default value, which the user can change via the dropdown.
NCBI's priority feedback
This breaks our migration scripts so is required for deployment
QA Steps
Regarding Display Heading Level
- Create any book (except completed book of Word workflow)
- In the second step of book creation, notice the value in the field "Display heading level". The default value is "4", but it might be different based on the organisation and collection templates that this book belongs to.
- In the drop down list of "Display heading field", try to change the value to any of the strings as following: 1, 2, 3, 4, 5, 99, 1:nojava, 2:nojava, 3:nojava,
<blank>
. - You can change this value again in the section "Landing Page: Table of Contents" in:
- Book Settings,
- "Book Templates" of the organisation and
- "Collection members' settings templates" if the collection is "Book Series Collection"
Regarding Expand heading level
- Create any book or go to an existing one (except completed book of Word workflow)
- Go to the 'Settings' of the book and in the section "Landing Page: Table of Contents", just below the "Display Heading Level", notice the value of "Expand heading level" The default value is "4", but it might be different based on the organisation and collection templates that this book belongs to.
- In the drop down list of "Expand heading level", change the value to any of the following: 1, 2, 3, 4 or
<blank>
. - You can change this value also in the section "Landing Page: Table of Contents" in:
- "Book Templates" of the organisation and
- "Collection members' settings templates" if the collection is "Book Series Collection"