Giter Club home page Giter Club logo

Comments (15)

dellus avatar dellus commented on May 24, 2024 1

It doesn't happen to me with alpha 369 Appimage on Puppylinux Fossapup.

from librecad.

dellus avatar dellus commented on May 24, 2024 1

From alpha 400 on Print Preview crashed LibreCAD for me too. In alpha 411 it didn't, Print Preview came up. But if I try to change the scale - again crash.
Clearing all settings in Application Preferences and running in a fresh Puppylinux live environment didn't help.

By the way, for not to open a new issue, -update current pen by active layer pen- also crashes LibreCAD.

from librecad.

dellus avatar dellus commented on May 24, 2024 1

Start 2.2.1-alpha 411 appimage.
Load an existing drawing or make a new one.
Open Print Preview. It starts as expected.
Try to set a different print scale. As soon as you select that scale LibreCAD exites.

The other issue, which doesn't belong into this thread:
In a new or existing drawing press the 'update current pen by active layer pen' button (double green triangle) sitting next to the standard pen selection tabs, introduced recently with the new pen palette dock widgets: - Exit.

from librecad.

perepujal avatar perepujal commented on May 24, 2024 1

Just checking the validity of penPaletteWidget seems to do the job for me, no more crashes:

diff --git a/librecad/src/ui/qg_actionhandler.cpp b/librecad/src/ui/qg_actionhandler.cpp
index 644829a9..0cd71c1e 100644
--- a/librecad/src/ui/qg_actionhandler.cpp
+++ b/librecad/src/ui/qg_actionhandler.cpp
@@ -2018,7 +2018,8 @@ void QG_ActionHandler::slotPenCopy(){
 
 void QG_ActionHandler::slotPenSyncFromLayer(){
     LC_PenPaletteWidget* penPaletteWidget  = QC_ApplicationWindow::getAppWindow()->getPenPaletteWidget();
-    penPaletteWidget->updatePenToolbarByActiveLayer();
+    if (penPaletteWidget != nullptr)
+      penPaletteWidget->updatePenToolbarByActiveLayer();
 }
 
 void QG_ActionHandler::set_view(RS_GraphicView* gview)

from librecad.

dongxuli-concord-design avatar dongxuli-concord-design commented on May 24, 2024

Bad regression.

from librecad.

dellus avatar dellus commented on May 24, 2024

I should have been more detailed: it's on Fossapup bare, which is a very cut-down version and does not contain QT, maybe that's the key.

from librecad.

GSStnb avatar GSStnb commented on May 24, 2024

More testing of alpha 369 Appimage on virtual machines...
works on MX Linux (VM)
works on a Debian GNU/Linux trixie/sid (VM)

Time to rebuild my desktop |-p

from librecad.

GSStnb avatar GSStnb commented on May 24, 2024

One more test: I wiped my LibreCAD configuration and started with a new config and it worked fine - Print Preview appeared as it should.

from librecad.

dongxuli-concord-design avatar dongxuli-concord-design commented on May 24, 2024

Might be several distinct issues.

Let's handle one by one. Any simple steps for me to reproduce the crash?

from librecad.

dxli avatar dxli commented on May 24, 2024

@dellus ,

Cannot reproduce the ‘'update current pen by active layer pen' issue.

from librecad.

sand1024 avatar sand1024 commented on May 24, 2024

The other issue, which doesn't belong into this thread:
In a new or existing drawing press the 'update current pen by active layer pen' button (double green triangle) sitting next to the standard pen selection tabs, introduced recently with the new pen palette dock widgets: - Exit.

Do you experience such issue with visible or hidden PenPalette widget?

Actually, that action relies on functionality provided by the widget, and as far as I can see by one of intermediate commit, if the widget is closed it may work incorrectly.

I think in several days I'll commit a fix for this,

from librecad.

dongxuli-concord-design avatar dongxuli-concord-design commented on May 24, 2024

@sand1024

Could review the widget layout also?
Currently, they might be too wide for people with narrow screens (1920 or less).

I have moved the pen pallet and layer tree into tabs, but more work may be needed here.

Regards!

from librecad.

dellus avatar dellus commented on May 24, 2024

In the 2.2.1-alpha 412 appimage changing the scale in Print Preview doesn't crash LC anymore, but has no effect at all.

The other issue, 'update current pen by active layer pen': Pen Palette and Layer tree are indeed disabled. But they seem not to exist in this 412 appimage, I cannot find any tabs or settings to enable them.

from librecad.

dongxuli-concord-design avatar dongxuli-concord-design commented on May 24, 2024

Need a setting in

ExtraToolbars:

UsePenPallet=1

from librecad.

perepujal avatar perepujal commented on May 24, 2024

It crashes for me in commit 10dd2da, start LibreCAD, no need to open an existing drawing, click the "Update Current Pen by Active Layer' Pen"

I've run LC on valgrind and attached the output in case it helps.
HTH
Pere
Librecad-Valgrind.log

from librecad.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.