We have moved to Github. Please open tickets there.

Opened 2 years ago

Closed 16 months ago

Last modified 16 months ago

#3509 closed Enhancement (fixed)

Update elgg_register_menu_item documentation for confirmations

Reported by: vazco Owned by:
Priority: normal Milestone: Elgg 1.8.4
Component: Core Version: 1.8 Beta
Severity: minor Keywords:
Cc: brett@… Difficulty: easy

Description

Currently there's no way to force confirmation with use of this function: elgg_register_menu_item

It would be nice if the link could display javascript confirmation before performing click, when we pass a parameter:

'confirmation' => 'Please confirm you want to launch this link'

Change History (5)

comment:1 Changed 2 years ago by vazco

heh, I can see it's already implemented... It would be good to add this to documentation of elgg_register_menu_item function as well.

comment:2 Changed 2 years ago by cash

  • Component changed from Core to Documentation
  • Difficulty set to easy
  • Milestone changed from Needs Review to Elgg 1.8.x
  • Summary changed from elgg_register_menu_item don't support confirmations to Update elgg_register_menu_item documentation for confirmations
  • Type changed from Defect to Enhancement

'confirm' is the key.

comment:3 Changed 16 months ago by cash

  • Component changed from Documentation to Core
  • Milestone changed from Elgg 1.8.x to Elgg 1.8.4

comment:4 Changed 16 months ago by Cash Costello

  • Resolution set to fixed
  • Status changed from new to closed

Fixes #3509 added documentation to menu system about confirm links

Changeset: 1971bda0d74afa534b0149a7533ce04f3890ffec

comment:5 Changed 16 months ago by Cash Costello

Fixes #3509 added documentation to menu system about confirm links

Changeset: 1971bda0d74afa534b0149a7533ce04f3890ffec

Note: See TracTickets for help on using tickets.