> Attached is an updated patch, which is even uglier because it tries to > display 'Embedded:/filename' in the graphic dialog when the file is > embedded. As you can imagine, back and forth translations are needed.
I am actually waiting for someone to say ' this is way too ugly. Why do not you merge DocFileName and EmbeddedFile and let InsetGraphicsParams handle this by itself?' The actual problem is then, due to the way param2string and InsetGraphicsParams::write() is implemented, changing InsetGraphicsParams implies file format change. I actually like the idea to add inzipName etc to DocFileName because Embedding needs to be handled in all places where DocFileName is used. Then, embedding status will be saved along with insets, and the manifest section will only have extra embedded files. Sounds like a plan? Bo
