View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000819 | FreeCAD | Bug | public | 2012-08-30 10:27 | 2016-07-23 10:40 |
Reporter | pperisin | Assigned To | |||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 0.12 | ||||
Fixed in Version | 0.17 | ||||
Summary | 0000819: Faces colors lost again (sorry) | ||||
Description | Hi, I have noticed that shapes loose colors on delete. So if you have shape with single color, or with color-per-face, they are lost when you delete object and press undo to undelete object. Undeleted one has default color | ||||
Additional Information | Win7 64bit. Make box, add color to it (per face, or per solid), delete object, press undo to undelete - color lost | ||||
Tags | No tags attached. | ||||
FreeCAD Information | |||||
related to | 0000020 | closed | UndoRedo for Gui Elements |
|
Undo/redp only works on App level but not on Gui level. Thus, face colors cannot be restored any more. See also 0000020 |
|
OK. I will suggest something that even to me seems little stupid, but it might be worth considering in order to prevent users from loosing information (their work). Since it is not finished, maybe additional check can be made before object is deleted - check if object has color-per-face info. if it does, show message that says objects have color-per-face info that will be lost permanently delete. I know it is a little stupid, especially if you have a lot of objects that can be deleted when you import STEP file and the message would appear every time you click delete (you could select those you want to delete and press del only once, to get only one message). color-per-model info is not so important, since it is easy to restore it. Regards, Petar |
FreeCAD: master f0e00311 2016-06-18 21:03:13 Details Diff |
prepare view provider for undo/redo |
Affected Issues 0000819, 0001118, 0002054 |
|
mod - src/App/Application.cpp | Diff File | ||
mod - src/App/CMakeLists.txt | Diff File | ||
mod - src/App/Document.cpp | Diff File | ||
mod - src/App/Document.h | Diff File | ||
mod - src/App/DocumentObject.cpp | Diff File | ||
mod - src/App/DocumentObject.h | Diff File | ||
add - src/App/TransactionalObject.cpp | Diff File | ||
add - src/App/TransactionalObject.h | Diff File | ||
mod - src/App/Transactions.cpp | Diff File | ||
mod - src/App/Transactions.h | Diff File | ||
mod - src/Gui/Application.cpp | Diff File | ||
mod - src/Gui/CMakeLists.txt | Diff File | ||
add - src/Gui/TransactionObject.cpp | Diff File | ||
add - src/Gui/TransactionObject.h | Diff File | ||
mod - src/Gui/ViewProvider.cpp | Diff File | ||
mod - src/Gui/ViewProvider.h | Diff File | ||
mod - src/Gui/ViewProviderDocumentObject.cpp | Diff File | ||
mod - src/Gui/ViewProviderDocumentObject.h | Diff File | ||
FreeCAD: master 9ec06b47 2016-06-20 12:50:21 Details Diff |
prepare view provider for undo/redo |
Affected Issues 0000819, 0001118, 0002054 |
|
mod - src/App/Document.cpp | Diff File | ||
mod - src/App/Document.h | Diff File | ||
mod - src/App/TransactionalObject.cpp | Diff File | ||
mod - src/App/TransactionalObject.h | Diff File | ||
mod - src/App/Transactions.h | Diff File | ||
mod - src/Gui/Document.cpp | Diff File | ||
mod - src/Gui/Document.h | Diff File | ||
mod - src/Gui/TransactionObject.cpp | Diff File | ||
mod - src/Gui/ViewProvider.cpp | Diff File | ||
mod - src/Gui/ViewProvider.h | Diff File | ||
mod - src/Gui/ViewProviderDocumentObject.cpp | Diff File | ||
mod - src/Gui/ViewProviderDocumentObject.h | Diff File | ||
FreeCAD: master 0f2db906 2016-06-20 14:00:47 Details Diff |
prepare view provider for undo/redo |
Affected Issues 0000819, 0001118, 0002054 |
|
mod - src/App/Document.cpp | Diff File | ||
mod - src/App/Transactions.cpp | Diff File | ||
mod - src/App/Transactions.h | Diff File | ||
mod - src/Gui/ViewProviderDocumentObject.cpp | Diff File | ||
FreeCAD: master f2bad459 2016-06-20 21:58:00 Details Diff |
+ fix order of removed objects in transaction list + add assert to check for obvious error + removed unneeded method |
Affected Issues 0000819, 0001118, 0002054 |
|
mod - src/App/Transactions.cpp | Diff File | ||
mod - src/Gui/Document.cpp | Diff File | ||
mod - src/Gui/Document.h | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2012-08-30 10:27 | pperisin | New Issue | |
2012-08-31 09:07 | wmayer | Note Added: 0002333 | |
2012-08-31 09:07 | wmayer | Note Edited: 0002333 | |
2012-08-31 09:07 | wmayer | Relationship added | related to 0000020 |
2012-08-31 10:48 | pperisin | Note Added: 0002336 | |
2012-08-31 10:49 | pperisin | Note Edited: 0002336 | |
2012-12-20 20:25 |
|
Status | new => assigned |
2012-12-20 20:25 |
|
Assigned To | => Jriegel |
2012-12-20 20:25 |
|
Status | assigned => confirmed |
2016-07-23 10:39 | wmayer | Changeset attached | => FreeCAD Master master f0e00311 |
2016-07-23 10:40 | wmayer | Changeset attached | => FreeCAD Master master 9ec06b47 |
2016-07-23 10:40 | wmayer | Changeset attached | => FreeCAD Master master 0f2db906 |
2016-07-23 10:40 | wmayer | Changeset attached | => FreeCAD Master master f2bad459 |
2016-07-23 10:40 | wmayer | Status | confirmed => closed |
2016-07-23 10:40 | wmayer | Resolution | open => fixed |
2016-07-23 10:40 | wmayer | Fixed in Version | => 0.17 |