[TD]fix hidden line regression,
This commit is contained in:
committed by
Chris Hennes
parent
7ba8b1f2ba
commit
c1df1c73a0
@@ -60,7 +60,6 @@ QGIEdge::QGIEdge(int index) :
|
||||
// not FreeCAD cosmetic lines
|
||||
void QGIEdge::setCosmetic(bool state)
|
||||
{
|
||||
// Base::Console().Message("QGIE::setCosmetic(%d)\n", state);
|
||||
isCosmetic = state;
|
||||
if (state) {
|
||||
setWidth(0.0);
|
||||
@@ -69,15 +68,9 @@ void QGIEdge::setCosmetic(bool state)
|
||||
|
||||
void QGIEdge::setHiddenEdge(bool b) {
|
||||
isHiddenEdge = b;
|
||||
if (b) {
|
||||
m_pen.setStyle(getHiddenStyle());
|
||||
} else {
|
||||
m_pen.setStyle(Qt::SolidLine);
|
||||
}
|
||||
}
|
||||
|
||||
void QGIEdge::setPrettyNormal() {
|
||||
// Base::Console().Message("QGIE::setPrettyNormal()\n");
|
||||
if (isHiddenEdge) {
|
||||
m_pen.setColor(getHiddenColor());
|
||||
return;
|
||||
@@ -91,14 +84,6 @@ QColor QGIEdge::getHiddenColor()
|
||||
return PreferencesGui::getAccessibleQColor(fcColor.asValue<QColor>());
|
||||
}
|
||||
|
||||
Qt::PenStyle QGIEdge::getHiddenStyle()
|
||||
{
|
||||
//Qt::PenStyle - NoPen, Solid, Dashed, ...
|
||||
//Preferences::General - Solid, Dashed
|
||||
// Dashed lines should use ISO Line #2 instead of Qt::DashedLine
|
||||
Qt::PenStyle hidStyle = static_cast<Qt::PenStyle> (Preferences::getPreferenceGroup("General")->GetInt("HiddenLine", 0) + 1);
|
||||
return hidStyle;
|
||||
}
|
||||
|
||||
double QGIEdge::getEdgeFuzz() const
|
||||
{
|
||||
@@ -123,16 +108,17 @@ QPainterPath QGIEdge::shape() const
|
||||
void QGIEdge::mouseDoubleClickEvent(QGraphicsSceneMouseEvent *event)
|
||||
{
|
||||
Q_UNUSED(event)
|
||||
QGIView *parent = dynamic_cast<QGIView *>(parentItem());
|
||||
auto* parent = dynamic_cast<QGIView *>(parentItem());
|
||||
if (parent && parent->getViewObject() && parent->getViewObject()->isDerivedFrom<TechDraw::DrawViewPart>()) {
|
||||
TechDraw::DrawViewPart *baseFeat = static_cast<TechDraw::DrawViewPart *>(parent->getViewObject());
|
||||
auto* baseFeat = static_cast<TechDraw::DrawViewPart *>(parent->getViewObject());
|
||||
std::vector<std::string> edgeName(1, DrawUtil::makeGeomName("Edge", getProjIndex()));
|
||||
|
||||
Gui::Control().showDialog(new TaskDlgLineDecor(baseFeat, edgeName));
|
||||
}
|
||||
}
|
||||
|
||||
void QGIEdge::setLinePen(QPen linePen)
|
||||
void QGIEdge::setLinePen(const QPen& linePen)
|
||||
{
|
||||
m_pen = linePen;
|
||||
}
|
||||
|
||||
|
||||
@@ -47,14 +47,14 @@ public:
|
||||
|
||||
void setCosmetic(bool state);
|
||||
void setHiddenEdge(bool b);
|
||||
bool getHiddenEdge() { return(isHiddenEdge); }
|
||||
bool getHiddenEdge() const { return(isHiddenEdge); }
|
||||
void setSmoothEdge(bool b) { isSmoothEdge = b; }
|
||||
bool getSmoothEdge() { return(isSmoothEdge); }
|
||||
bool getSmoothEdge() const { return(isSmoothEdge); }
|
||||
void setPrettyNormal() override;
|
||||
|
||||
double getEdgeFuzz() const;
|
||||
|
||||
void setLinePen(QPen isoPen);
|
||||
void setLinePen(const QPen& isoPen);
|
||||
|
||||
void setSource(TechDraw::SourceType source) { m_source = source; }
|
||||
TechDraw::SourceType getSource() const { return m_source;}
|
||||
@@ -65,15 +65,15 @@ protected:
|
||||
|
||||
bool multiselectEligible() override { return true; }
|
||||
|
||||
QColor getHiddenColor();
|
||||
|
||||
private:
|
||||
int projIndex; //index of edge in Projection. must exist.
|
||||
|
||||
bool isCosmetic;
|
||||
bool isHiddenEdge;
|
||||
bool isSmoothEdge;
|
||||
QColor getHiddenColor();
|
||||
Qt::PenStyle getHiddenStyle();
|
||||
|
||||
private:
|
||||
TechDraw::SourceType m_source{TechDraw::SourceType::GEOMETRY};
|
||||
};
|
||||
|
||||
|
||||
Reference in New Issue
Block a user