Currently, the only event operation (delete) has a very different markup than the operations of conditions and actions.
Would be nice if we could align this.

CommentFileSizeAuthor
#1 2059621_align_event_operations_markup.patch1.25 KBdasjo

Comments

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
commented
StatusFileSize
new 2059621_align_event_operations_markup.patch1.25 KB

the attached patch changes the operations markup to

<td>
  <ul class="rules-operations action-links rules_rule_event">
    <li class="delete_event first last"><a href="...delete_url...">delete</a></li>
  </ul>
</td>
dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
commented
Status: Active » Needs review
dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
commented

btw. in the future we might want to be able to edit events and their settings as well (should be handled in a separate issue)

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs review » Reviewed & tested by the community

Thanks, looks good - will commit later.

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Reviewed & tested by the community » Fixed

Thanks.

Status: Fixed » Closed (fixed)

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