Re: Workarea not updated after deleting display math, on current master

2024-04-17 Thread Jean-Marc Lasgouttes

Le 17/04/2024 à 04:19, Scott Kostyshak a écrit :

Does this work?


Yes, works well here! Thanks.


Thanks for testing, it is in :)

JMarc

--
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel


Re: Workarea not updated after deleting display math, on current master

2024-04-16 Thread Scott Kostyshak
On Wed, Apr 17, 2024 at 12:04:24AM GMT, Jean-Marc Lasgouttes wrote:
> Le 12/04/2024 à 18:16, Scott Kostyshak a écrit :
> > To reproduce:
> > 
> > 1. Open the attached document.
> > 2. Select all.
> > 3. Backspace (to delete everything).
> > 
> > I get the attached screenshot.
> > 
> > Can anyone reproduce?
> 
> Yes!
> 
> > By the way, my secret goal with this batch of bug reports on current
> > master is to ruin JMarc's Friday evening/weekend :)
> > However, I think he has healthy work habits so hopefully he doesn't
> > check his emails.
> 
> So instead I spent my evening trying to fix what you just broke (the bug
> would not have existed if nobody had tried to trigger it).

;) 

> Does this work?

Yes, works well here! Thanks.

Scott


signature.asc
Description: PGP signature
-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel


Re: Workarea not updated after deleting display math, on current master

2024-04-16 Thread Jean-Marc Lasgouttes

Le 12/04/2024 à 18:16, Scott Kostyshak a écrit :

To reproduce:

1. Open the attached document.
2. Select all.
3. Backspace (to delete everything).

I get the attached screenshot.

Can anyone reproduce?


Yes!


By the way, my secret goal with this batch of bug reports on current
master is to ruin JMarc's Friday evening/weekend :)
However, I think he has healthy work habits so hopefully he doesn't
check his emails.


So instead I spent my evening trying to fix what you just broke (the bug 
would not have existed if nobody had tried to trigger it).


Does this work?

JMarc


From 05e5be4926ecd79f8263beae895e79b2f57bb61c Mon Sep 17 00:00:00 2001
From: Jean-Marc Lasgouttes 
Date: Tue, 16 Apr 2024 23:55:24 +0200
Subject: [PATCH] Always repaint the gray area below main inset

Now that SingleParUpdate does not always lead to a full screen update
when the height of the paragraph changes (see new behavior of
updateMatrics(bool)), it is necessary to make sure that the grey area
below the main page is always repainted.
---
 src/BufferView.cpp | 11 +++
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/BufferView.cpp b/src/BufferView.cpp
index 7abaea0d40..404e38cb5b 100644
--- a/src/BufferView.cpp
+++ b/src/BufferView.cpp
@@ -3711,7 +3711,13 @@ void BufferView::draw(frontend::Painter & pain, bool paint_caret)
 		// Draw everything.
 		tm.draw(pi, 0, y);
 
-		// and possibly grey out below
+		break;
+	}
+	LYXERR(Debug::PAINTING, (pain.isNull() ? "\t\t --- END NODRAW ---"
+	: "\t\t *** END DRAWING ***"));
+
+	// Possibly grey out below
+	if (d->update_strategy_ != NoScreenUpdate) {
 		pair lastpm = tm.last();
 		int const y2 = lastpm.second->bottom();
 
@@ -3720,10 +3726,7 @@ void BufferView::draw(frontend::Painter & pain, bool paint_caret)
 ? Color_background : Color_bottomarea;
 			pain.fillRectangle(0, y2, width_, height_ - y2, color);
 		}
-		break;
 	}
-	LYXERR(Debug::PAINTING, (pain.isNull() ? "\t\t --- END NODRAW ---"
-	: "\t\t *** END DRAWING ***"));
 
 	// The scrollbar needs an update.
 	// FIXME: does it always? see ticket #11947.
-- 
2.40.1

-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel


Re: Workarea not updated after deleting display math, on current master

2024-04-12 Thread Scott Kostyshak
On Fri, Apr 12, 2024 at 06:57:25PM GMT, Jean-Marc Lasgouttes wrote:
> I do check the email but I am in week end out of home so nothing will be done 
> :)
> 
> JMarc 

Enjoy the weekend! :)

Scott


> Le 12 avril 2024 18:16:34 GMT+02:00, Scott Kostyshak  a 
> écrit :
> >To reproduce:
> >
> >1. Open the attached document.
> >2. Select all.
> >3. Backspace (to delete everything).
> >
> >I get the attached screenshot.
> >
> >Can anyone reproduce?
> >
> >By the way, my secret goal with this batch of bug reports on current
> >master is to ruin JMarc's Friday evening/weekend :)
> >However, I think he has healthy work habits so hopefully he doesn't
> >check his emails.
> >
> >Scott


signature.asc
Description: PGP signature
-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel


Re: Workarea not updated after deleting display math, on current master

2024-04-12 Thread Jean-Marc Lasgouttes
I do check the email but I am in week end out of home so nothing will be done :)

JMarc 

Le 12 avril 2024 18:16:34 GMT+02:00, Scott Kostyshak  a écrit 
:
>To reproduce:
>
>1. Open the attached document.
>2. Select all.
>3. Backspace (to delete everything).
>
>I get the attached screenshot.
>
>Can anyone reproduce?
>
>By the way, my secret goal with this batch of bug reports on current
>master is to ruin JMarc's Friday evening/weekend :)
>However, I think he has healthy work habits so hopefully he doesn't
>check his emails.
>
>Scott
-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel