Opened 9 years ago

Closed 4 years ago

#544 closed enhancement (wontfix)

Patch to add a ID to a Image (InsertImage Popup)

Reported by: buz Owned by: gogo
Priority: normal Milestone:
Component: Xinha Core Version:
Severity: normal Keywords: image img id patch
Cc:

Description

The patch adds a ID field for the default image insert.

Attachments (1)

xinha.patch (1.9 KB) - added by anonymous 9 years ago.
The patch is against a few days old Xinha SVN version. The revision in the file is bogus (as it lives in our own SVN).

Download all attachments as: .zip

Change History (10)

Changed 9 years ago by anonymous

The patch is against a few days old Xinha SVN version. The revision in the file is bogus (as it lives in our own SVN).

comment:1 Changed 9 years ago by niko

why is this needed?

comment:2 Changed 9 years ago by gocher

  • Summary changed from Patch to Patch to add a ID to a Image (InsertImage Popup)

comment:3 Changed 9 years ago by buz

We added this for use with certain CMS (ezpublish in our case) that use the ID to reference images internally.

It might be of some use to others, or maybe not. I've since written my own, much more ezpublish CMS specific extension so feel free to disregard this if you don't see any use for it.

comment:4 Changed 9 years ago by niko

we have two options to implement this:

  • using a config.insertImageShowsId = true (like config.makeLinkShowsTarget)
  • you write a new plugin based on the insertImage that uses a new dialog with ID

i would prefer the first option... any other oppinions?

comment:5 Changed 9 years ago by gogo

First option niko - config.ImageManager.promptID or something. Should be off by default since not many people will need it, but still useful.

comment:6 Changed 8 years ago by anonymous

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

lkjlkj[
{{{

==



}}}
]

comment:7 Changed 8 years ago by anonymous

  • Resolution fixed deleted
  • Status changed from closed to reopened

Bloody anonymous.

comment:8 Changed 8 years ago by gogo

  • Keywords patch added

comment:9 Changed 4 years ago by gogo

  • Resolution set to wontfix
  • Status changed from reopened to closed

Closing this, things have changed and nobody else has asked, so either it's done or nobody actually needs it (I kinda think it's not suitable to ask end users for an "ID" anyway).

Note: See TracTickets for help on using tickets.