heikoklare / ifk-autocad-tools Goto Github PK

Currently, a framed text can be created for a single text box. However, you sometimes want to have a text frame around multiple text boxes. We could add a function that supports this feature.

Currently, creating a frame for a text box is a one-shot operation. Changing the text requires a manual update of the frame or a reexecution of the command to generate the frame.
We could attach an object (xref) to the text box to get notified whenever the text changes. This event can be used to update the according frame whenever the text changes.

The code in this repository has evolved over several years and over several steps in learning software engineering practices. There are several design flaws, interwoven UI with core functionality, magic values spreat over the code, code duplications etc.
Thus, this is an ongoing issue to solve.

Currently, the text frame attached to texts has a fixed size, i.e., a fixed padding between text and frame. We can allow customization of this size. There should be a default value set when creating frame for a text box. However, there should be the option to enter a different value.

When moving a drawing area (copy or cut and paste) to another drawing, the drawing becomes broken. Not only that the moved drawing area cannot be used properly, there are even workflows that lead to a crash of the complete application:

1. Create a drawing area
2. Copy the drawing area into a new drawing
3. Create another drawing area in the new drawing (both will exist then)
4. Save and close the new drawing
5. Reload the drawing (a message telling you that the drawing area cannot be found will show up)
6. Create a new drawing area
7. An access violation error will occur.

Copying drawing areas to other drawing should either be properly supported or, more likely as this functionality is not really needed, trying to copy them should be rejected.