[PAGES-1007] Norsu pages app should not allow space in name Created: 30/Jan/23 Updated: 04/Apr/23 Resolved: 15/Feb/23 |
|
| Status: | Closed |
| Project: | Magnolia pages module |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Neutral |
| Reporter: | Christopher Zimmermann | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Σ Remaining Estimate: | Not Specified | Remaining Estimate: | Not Specified |
| Σ Time Spent: | Not Specified | Time Spent: | Not Specified |
| Σ Original Estimate: | Not Specified | Original Estimate: | Not Specified |
| Attachments: |
|
|||||||||||||||||||||||||
| Issue Links: |
|
|||||||||||||||||||||||||
| Sub-Tasks: |
|
|||||||||||||||||||||||||
| Template: | ||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
|||||||||||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
|||||||||||||||||||||||||
| Date of First Response: | ||||||||||||||||||||||||||
| Epic Link: | Norsu: After TPR | |||||||||||||||||||||||||
| Team: | ||||||||||||||||||||||||||
| Description |
|
Norsu Pages app has the same "Add Page" dialog as the JCR pages app. But when you supply a string with spaces for the "Page name" field in JCR then the spaces are replacd with dashes, whereas in the Norsu app the spaces are not removed. The Norsu Pages app should not create names with spaces as these result in unattractive URLS in the browser where all the spaces are replaced with "%20". I would expect the Norsu dialog to work like the JCR dialog does. Other solutions could be proposed too. A solution sugggested in the comments would be to use the new nameField ingtroduced in NORSU-912. Doing so would require additional decision making. Ultimiately the decision on what to do belongs to AuthorX team. See attached images.
|
| Comments |
| Comment by Dominik Maslanka [ 06/Feb/23 ] |
|
case got covered within https://jira.magnolia-cms.com/browse/NORSU-912 |
| Comment by Christopher Zimmermann [ 06/Feb/23 ] |
|
Have you checked with AuthorX and Design team? |
| Comment by Christopher Zimmermann [ 08/Feb/23 ] |
|
Re-opening because the problem is still existing and there is not an agreed upon solution. There is the possiblity that |
| Comment by Dominik Maslanka [ 15/Feb/23 ] |
|
After checking with ldelnevo and checking on SaaS Prod - I'm closing this ticket as use case got covered with introducing new Name + Slug Field Screen Recording 2023-02-15 at 11.59.16.mov |