[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: File Screen Recording 2023-02-15 at 11.59.16.mov     PNG File jcr-browser.png     PNG File norsu-browser.png    
Issue Links:
Relates
dependency
depends upon PAGES-1000 Enable the norsu name validator Closed
Sub-Tasks:
Key
Summary
Type
Status
Assignee
PAGES-1008 Implementation Sub-task To Do  
PAGES-1009 Review Sub-task To Do  
PAGES-1010 Pre-Integration QA Sub-task To Do  
PAGES-1011 QA Sub-task To Do  
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: Nucleus

 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 PAGES-1000 may solve it.

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

Generated at Mon Feb 12 06:24:14 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.