Unless the user knows to look in the browsers address bar there's no what of knowing which event type the user is about to create a series in.

A site builder is likely to know where to look but a content creator/editor wont.

screenshot

CommentFileSizeAuthor
#8 recurring_events-add_form_title-3116932-8-cache.patch4.4 KBowenbush
#8 interdiff-3-8.txt3 KBowenbush
#5 Screen Shot 2020-03-05 at 6.10.51 pm.png146.62 KBthe_glitch
#3 recurring_events-add_form_title-3116932-3.patch2.03 KBowenbush
Screen Shot 2020-03-01 at 8.00.15 pm.png137.02 KBthe_glitch

Comments

the_glitch created an issue. See original summary.

owenbush’s picture

owenbush commented

Thanks for the issue. I'll get this looked at.

owenbush’s picture

owenbush commented
StatusFileSize
new recurring_events-add_form_title-3116932-3.patch2.03 KB

Attached is a patch to add the event series type to the form title.

owenbush’s picture

owenbush commented
Status: Active » Needs review

Marking as needs review.

the_glitch’s picture

the_glitch commented
Component: User interface » Recurring Events (Main module)
StatusFileSize
new Screen Shot 2020-03-05 at 6.10.51 pm.png146.62 KB

Patch doesn't apply event type's title to add/edit form

screenshot

owenbush’s picture

owenbush commented

Can you confirm the patch actually applied at the composer level? Does it display on the add form? I see it doesn't display on the edit form so I'll get that addressed, but it should be displaying on the add form.

the_glitch’s picture

the_glitch commented

My mistake. Yes it does apply to the Add form.

owenbush’s picture

owenbush commented
StatusFileSize
new interdiff-3-8.txt3 KB
new recurring_events-add_form_title-3116932-8-cache.patch4.4 KB

Attached is a new patch to also update the title on edit and clone pages for event series. There is also an interdiff supplied.

mrpauldriver’s picture

mrpauldriver commented

This is working for me

owenbush’s picture

owenbush commented
Status: Needs review » Reviewed & tested by the community

Great, Thanks Paul. I'll mark this little guy as Reviewed and Tested by the Community and get it merged soon.

the_glitch’s picture

the_glitch commented

Yes patch #8 works. Thanks Owen.

  • owenbush committed 665d6d4 on 8.x-1.x 
    Issue #3116932 by owenbush, the_glitch, MrPaulDriver: Display event type...
owenbush’s picture

owenbush commented
Status: Reviewed & tested by the community » Fixed

This has been merged. Thanks to all that help test and raise the issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.