Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request:] Please remove "Themes" from the "Documents" folder. #780

Closed
milke opened this issue May 2, 2022 · 14 comments
Closed

Comments

@milke
Copy link

milke commented May 2, 2022

I've just discovered SQL Pro Studio and I find it to be an excellent tool. I'm in a trial period and I'm considering purchasing a lifetime license, which I'll probably do. But I have one very small request. It seems you put Themes folder in the (sandboxed) user's Documents folder, which then ends up in ~/Library/Containers/com.hankinsoft.osx.sqlprostudio/Data/Documents. However, that automatically adds a reference to it in the ~/Documents folder, see attached picture.

I really don't want to see anything in my ~/Documents I didn't put myself in there. Besides, themes don't really sound like something that should go into the Documents folder, Application Support (or something similar) sounds like much better solution.

Thanks for considering this and keep up the great work!

documents

@El-Ekim
Copy link

El-Ekim commented May 11, 2022

I agree with Mike. I tried to hide the folder Themes in
~/Library/Containers/com.hankinsoft.osx.sqlprostudio/Data/Documents
That does work but now I have an empty SQLProStudio folder in my Documents folder.
Any suggestions on how to hide that one as well?

@hankinsoft
Copy link
Owner

I can take a look into moving themes for a future build as it would make sense for those to be part of application support, but just as an FYI other items such as favourites can also be stored in the documents directory in certain contains.

@hankinsoft
Copy link
Owner

Decided not to move these at the moment as documents is a require location for them to be shared via iCloud 9(when using sqlpro on multiple systems).

@milke
Copy link
Author

milke commented Apr 17, 2023

Hi, I'm not really happy with the outcome 🙁

I don't really care about sharing themes or anything else on multiple systems and I believe there are other users feeling the same. Would it be possible to introduce some kind of preference switch for sharing on multiple systems (you can even leave it ON by default) and in case a user switches it OFF, move stuff to Application Support or wherever appropriate?

I understand this would introduce some complexity in your code, but it shouldn't really be a problem, as otherwise SQL Pro Studio messes with user's ~/Documents folder, even if one doesn't want that.

Thanks for taking this into consideration. Apart from that, great work!! 👍🏻

@m-thomson
Copy link

@hankinsoft

Apps should not write anything to the Documents folder without permission. Please remove this cruft - it's a terrible practice and nobody else does it. I have over 50 apps from Setapp currently installed and over 160 on my system in total. None of them do this except yours. Not one.

Seriously. Please. Allow the user to specify the location or use the appropriate ~/Library/... folder.

Related: #723

@milke
Copy link
Author

milke commented Jul 15, 2023

As a fellow developer, one of the customer feedbacks I hate the most (to the extent that I even try to completely ignore them if possible) is "this is a great application, but I will buy/purchase it only if you implement that_and_that…". Unfortunately, I ended up in a similar situation (as a customer) in this case. In my defence, I'm not asking for a feature, but for a change in behaviour, which I find to be a borderline bug; the application messes with some file location(s) where it has no business at all!

As stated in my initial post here, more than a year ago, I was damn serious about purchasing lifetime license for the product and I wouldn't even mind paying some upgrade costs from time to time. Instead, I opted for a monthly subscription, waiting to see how this issue will be resolved. I cancelled the subscription only after 2 (or was it maybe 3) months, because I effectively stopped using the application. All bells and whistles it brings were comfortably outweighed by my frustration every time I went into the ~/Documents folder (and I go there MANY times a day).

It's really a shame, since the application is great otherwise and the implementation of the change isn't complicated nor time consuming at all.

Related: #723

@hankinsoft
Copy link
Owner

I'll be away until late next week, but I'll re-evaluate this at that point.

@hankinsoft hankinsoft reopened this Jul 15, 2023
@El-Ekim
Copy link

El-Ekim commented Jul 16, 2023

Adobe, FL Studio, Native Instruments all adds a folder (with lots of subfolders) in the Documents folder. SQL Pro doing this is not unique. Happily I managed to hide them all.
It doesn't bother me now but I do like the luxury of being able to choose the folder location myself ;)
I love SQL Pro Studio and an extra folder in my Documents folder is not going to change that.

@hankinsoft
Copy link
Owner

My plan is to move the themes into a folder under the applications '/Library/Themes' folder. Does anyone have any concerns with that?

@milke
Copy link
Author

milke commented Jul 26, 2023

My plan is to move the themes into a folder under the applications '/Library/Themes' folder. Does anyone have any concerns with that?

Not me.

@El-Ekim
Copy link

El-Ekim commented Jul 28, 2023

Sounds great.

@m-thomson
Copy link

Thanks for taking this up @hankinsoft !

@hankinsoft
Copy link
Owner

A build has been submitted to setapp which should resolve this. A website build is of SQLPro Studio also available at https://sqlprostudio.s3.us-east-1.amazonaws.com/studio/SQLProStudio.2023.55.app.zip. The App Store build should be available sometime next week.

@hankinsoft
Copy link
Owner

The App Store version of SQLPro studio now includes this as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants