Tracker: Bugs

3 Does not remeber window dimension after reboot - ID: 3138596
Last Update: Comment added ( lstranger )

Using Lubuntu 10.10

If you close PCManFM and reopen it, it will remember its dimension (height X width). But if you change dimensions, close it and reboot, it will reopen at the old dimensions, not the new ones that you set. Is this a bug or am I doing something wrong. Same problem on different computers.


Pierre C ( pierre16 ) - 2010-12-16 06:43:54 PST

3

Closed

Fixed

Lonely Stranger

pcmanfm

1.0.1

Public


Comments ( 8 )

Date: 2012-08-13 05:04:23 PDT
Sender: lstranger

Just the note - ticket was about "change dimensions, close it and reboot"
and that behavior is fixed in Git but your case is "change dimensions and
then reboot" which is another case and that another case may need yet be
confirmed if it is ever requested and if it is then it should be discussed
with all users as it changes current behavior so therefore the user
expectations. So let it be as it is now. Ticket's issue is fixed. Thank you
everyone!


Date: 2012-08-13 03:05:39 PDT
Sender: lstranger

You're right about killing with the window is opened, I've missed that
case. But you should not worry about any excessive disk writing as much
config will be saved if and only if the user manually resize window size
(not counting moving, maximizing, or fullscreening it) and even that will
be done with a little timeout so if user do resize few times in few seconds
then config will be rebuilt only once.


Date: 2012-08-13 02:21:34 PDT
Sender: pcmanxProject AdminAccepting Donations

I have never fixed this one so I'm sure the bug is still there.
Fixing this issue is more complicated then it looks like.

When to save the window position?
1. If we do it everytime the window is moved, this causes excessive disk
writing.
2. If we do it when a window is closed, if the program is terminated with
all window opened, it won't be saved.
3. If we do it when the program terminates, this sounds reasonable but
cannot be implemented. When the user logout the desktop session or reboot,
pcmanfm is terminated by SIGKILL generated by disconnecting from X. It's
SIGKILL, not SIGTERM and there is no way to intercept that signal.
4. If we utilize X11 session management and save it when X session logout,
this requires XSM support and it only work for currently active desktop
session. Sessions of inactive users are not handled by XSM. Besides,
lxsession and other WMs has no XSM support.
5. If we try to interact with session manager, such as register ourself via
gnome-session dbus interface, it's possible to get logout signal under
gnome, but it's gnome-only.

To sum up, there is no reliable and efficient way to do it while it's easy
to do in Windows since Windows 95. Unfortunately, we're using X11.


Date: 2012-08-12 17:26:09 PDT
Sender: lstranger

We hope this issue is completely fixed in the Git repository recently.
Check it out when it's possible for you and if the bug still exists then
reopen the report, please.
Thank you very much.


Date: 2012-05-26 19:28:39 PDT
Sender: pcmanxProject AdminAccepting Donations

If the program is killed, the values are not written to config files.
Otherwise, it saves the values in config files on program termination.
We need to find out a better way to do this.


Date: 2011-11-18 11:27:51 PST
Sender: pcmanxProject AdminAccepting Donations

Version 1.0 is in feature freeze and string freeze now.
Let's fix this later in 1.5.


Date: 2010-12-22 09:22:58 PST
Sender: pierre16

Correction:
It seems to use and load dimensions from lubuntu.conf (works if changed
manually) but fails to save new dimensions there upon quitting.


Date: 2010-12-22 09:10:52 PST
Sender: pierre16

Did more checking...

Other window positions are remembered. So it is a PCManFM problem. It
appears that it does not save to pcmanfm.conf:
win_width=?
win_height=?
splitter_pos=?

AND does not load any new dimensions either if you enter them manually.

Should be an easy fix. Thanks for the great work. It is a very good piece
of software!


Attached File

No Files Currently Attached

Changes ( 19 )

Field Old Value Date By
status_id Open 2012-08-13 03:05:39 PDT lstranger
close_date - 2012-08-13 03:05:39 PDT lstranger
allow_comments 1 2012-08-13 03:05:39 PDT lstranger
allow_comments 0 2012-08-13 02:21:34 PDT pcmanx
close_date 2012-08-12 17:26 2012-08-13 02:21:34 PDT pcmanx
status_id Closed 2012-08-13 02:21:34 PDT pcmanx
resolution_id Accepted 2012-08-12 17:26:09 PDT lstranger
close_date - 2012-08-12 17:26:09 PDT lstranger
allow_comments 1 2012-08-12 17:26:09 PDT lstranger
status_id Open 2012-08-12 17:26:09 PDT lstranger
assigned_to pcmanx 2012-08-10 10:57:06 PDT lstranger
category_id None 2012-07-18 06:13:03 PDT lstranger
resolution_id None 2012-07-17 13:54:06 PDT lstranger
artifact_group_id patches 2012-07-16 11:44:53 PDT lstranger
priority 2 2012-05-26 13:21:13 PDT lstranger
priority 4 2012-05-26 13:19:35 PDT lstranger
assigned_to nobody 2011-11-18 11:27:59 PST pcmanx
artifact_group_id None 2011-11-18 11:27:51 PST pcmanx
priority 5 2011-11-18 11:27:51 PST pcmanx