search

kmwallio / ThiefMD

The markdown editor worth stealing. Inspired by Ulysses, based on code from Quilter
https://thiefmd.com/
GNU General Public License v3.0
150 stars 8 forks source link

new sheet on open #159

Open g33kdad opened 2 years ago

g33kdad commented 2 years ago

Title: ThiefMD doesnt respect the "start with new empty sheet on launch" preference setting

Comment: ThiefMD opens with the last edited sheet open Using: Flatpak and RPM Desktop Environment: GNOME OS: OpenSuse Tumbleweed I am using/not using Wayland: this occurs in both X.org and Wayland

Steps:
1. Open ThiefMD
2. Set "start with new empty sheet on launch" preference
3. open sheet and edit
4. close ThiefMD
5. open ThiefMD again, expecting a blank page; find last edited document still open
kmwallio commented 2 years ago

Thanks for reporting this, I'll take a look.

g33kdad commented 2 years ago

I would like to add, I have tried the flatpak on Fedora SilverBlue 35, and the problem is present there, too.

Please feel free to let me know if I can help by providing logs or debug outputs or anything.

Cheers,

M

kmwallio commented 2 years ago

Can you share a screenshot of the Editor Preferences page? Are the folders in the library in the /home/user directory, or on another disk?

For capturing logs, you can run:

G_MESSAGES_DEBUG=all com.github.kmwallio.thiefmd >output.log 2>&1

or

G_MESSAGES_DEBUG=all flatpak run com.github.kmwallio.thiefmd >output.log 2>&1

I think I may have found the issue, and will hopefully have a fix out over the weekend. There's a chance we do a redraw where the setting isn't checked (the redraw loads the last known file), but I'm unable to repro it myself to confirm.

g33kdad commented 2 years ago

I will try to get those things this weekend.

M

On Fri, Jan 21, 2022, 11:50 AM Miles Wallio @.***> wrote:

Can you share a screenshot of the Editor Preferences page? Are the folders in the library in the /home/user directory, or on another disk?

For capturing logs, you can run:

G_MESSAGES_DEBUG=all com.github.kmwallio.thiefmd >output.log 2>&1

or

G_MESSAGES_DEBUG=all flatpak run com.github.kmwallio.thiefmd >output.log 2>&1

I think I may have found the issue, and will hopefully have a fix out over the weekend. There's a chance we do a redraw where the setting isn't checked (the redraw loads the last known file), but I'm unable to repro it myself to confirm.

— Reply to this email directly, view it on GitHub https://github.com/kmwallio/ThiefMD/issues/159#issuecomment-1018811941, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACODOZRJ5AHZORUH3BTIRTTUXG2JLANCNFSM5KEMF2HA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

kmwallio commented 2 years ago

I will try to get those things this weekend.

Thanks! No need to rush. Trying to step through this highlighted some logic and timing changes I should probably adjust.

g33kdad commented 2 years ago

here is the screenshot of the pref. pane. the library is in a directory under /home/user. fwiw, it is a directory that is synced to my Nextcloud server.

I will run the app with the debug message as you suggested in your email and then message you with those results, as well.

cheers,

m

On Fri, Jan 21, 2022 at 12:30 PM Miles Wallio @.***> wrote:

I will try to get those things this weekend.

Thanks! No need to rush. Trying to step through this highlighted some logic and timing changes I should probably adjust.

— Reply to this email directly, view it on GitHub https://github.com/kmwallio/ThiefMD/issues/159#issuecomment-1018837330, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACODOZXTYZGUU4JMLEGQFATUXG67JANCNFSM5KEMF2HA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

-- Matthew McGraw @.***

“ . . . we as a nation must undergo a radical revolution of values . . . When machines and computers, profit motives and property rights are considered more important than people, the giant triplets of racism, materialism, and militarism are incapable of being conquered.” [Dr. Martin Luther King, Jr., Riverside Church, April 4, 1967.]

g33kdad commented 2 years ago

here's the "output.log" file. i launched the app, removed the current library folder, added a different folder to the library, opened another file, and closed the app.

cheers,

m

On Sun, Jan 23, 2022 at 3:50 PM Matt McGraw @.***> wrote:

here is the screenshot of the pref. pane. the library is in a directory under /home/user. fwiw, it is a directory that is synced to my Nextcloud server.

I will run the app with the debug message as you suggested in your email and then message you with those results, as well.

cheers,

m

On Fri, Jan 21, 2022 at 12:30 PM Miles Wallio @.***> wrote:

I will try to get those things this weekend.

Thanks! No need to rush. Trying to step through this highlighted some logic and timing changes I should probably adjust.

— Reply to this email directly, view it on GitHub https://github.com/kmwallio/ThiefMD/issues/159#issuecomment-1018837330, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACODOZXTYZGUU4JMLEGQFATUXG67JANCNFSM5KEMF2HA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

-- Matthew McGraw @.***

“ . . . we as a nation must undergo a radical revolution of values . . . When machines and computers, profit motives and property rights are considered more important than people, the giant triplets of racism, materialism, and militarism are incapable of being conquered.” [Dr. Martin Luther King, Jr., Riverside Church, April 4, 1967.]

-- Matthew McGraw @.***

“ . . . we as a nation must undergo a radical revolution of values . . . When machines and computers, profit motives and property rights are considered more important than people, the giant triplets of racism, materialism, and militarism are incapable of being conquered.” [Dr. Martin Luther King, Jr., Riverside Church, April 4, 1967.]