Futura-Py / weather

Coming soon
MIT License
4 stars 1 forks source link

init: Basic GUI setup + upgrades #5

Closed im-coder-lg closed 1 year ago

im-coder-lg commented 1 year ago

I got some holidays, so let's do a basic GUI. I just want the searchbar to move a bit more up to house the deets. For now, it's simple, and I got some upgrades.

im-coder-lg commented 1 year ago

it's still WIP I just need to get the data and input it in the window. Will be doing that soon

im-coder-lg commented 1 year ago

/ping @Futura-Py/reviewers

HuyHung1408 commented 1 year ago

It works pretty well.

sumeshir26 commented 1 year ago

@im-coder-lg Cant test as of now, sorry...

im-coder-lg commented 1 year ago

Wait, I'll test from my end too.

im-coder-lg commented 1 year ago

Gosh, heck of an upgrade. Y'know, this is a good time for me to start learning classes lol. Thanks for the upgrade! And I got some more ideas, starting with adding the weather metrics to the GUI. I need help on this though.

im-coder-lg commented 1 year ago

Now for Linux, we can use snaps. Now, to the contrary(they say snaps are slow), snaps are effective for our use. They seem to do the job. We'll use those, then figure out flatpaks and then AppImages. Then maybe we can add DEB and RPM support. Among all of these options, I stick with Flatpak. Anyone else?

P. S. How do we add polls?

sumeshir26 commented 1 year ago

Now for Linux, we can use snaps. Now, to the contrary(they say snaps are slow), snaps are effective for our use. They seem to do the job. We'll use those, then figure out flatpaks and then AppImages. Then maybe we can add DEB and RPM support. Among all of these options, I stick with Flatpak. Anyone else?

Hmmm, how do we package?

P. S. How do we add polls?

Only avakaible for discussions.

rdbende commented 1 year ago

Anyone else?

I don't want to get into the frequent debate about how much better Flatpak is than Snaps and that Snaps aren't targeted for desktop apps, but I'd recommend Flatpak. It's easier to work with, it's much more popular and it's pre-installed on many distros unlike Snaps.

Here's how you'd add Tkinter to a Flatpak app: https://github.com/iwalton3/tkinter-standalone. And btw, I also have a work in progress package to make Tkinter integrate better with Linux systems and to make it better sandbox-able in a Flatpak ;))

Moosems commented 1 year ago

Now for Linux, we can use snaps. Now, to the contrary(they say snaps are slow), snaps are effective for our use. They seem to do the job. We'll use those, then figure out flatpaks and then AppImages. Then maybe we can add DEB and RPM support. Among all of these options, I stick with Flatpak. Anyone else?

What do you mean "snaps"?

rdbende commented 1 year ago

What do you mean "snaps"?

https://en.wikipedia.org/wiki/Snap_(software)

Moosems commented 1 year ago

What do you mean "snaps"?

https://en.wikipedia.org/wiki/Snap_(software)

Wasn't this the point of the PyInstaller builds?

rdbende commented 1 year ago

Wasn't this the point of the PyInstaller builds?

On Window$ and mac yes. Snaps are a Linux thing.

Moosems commented 1 year ago

Wasn't this the point of the PyInstaller builds?

On Window$ and mac yes. Snaps are a Linux thing.

Yeah I'm going to leave that to you linux people lol.

rdbende commented 1 year ago

On Linux, people don't just download random executables from the internet, like on win and mac, but instead use the package manager or the app store to install apps. Flatpaks and Snaps integrate nicely with most GUI app stores.

Moosems commented 1 year ago

But you dislike Snaps, so just use flatpaks?

rdbende commented 1 year ago

But you dislike Snaps, so just use flatpaks?

Yes, I dislike them, but that doesn't mean you can't implement it.

Moosems commented 1 year ago

The Flatpaks docs aren't too bad but it doesn't look like it takes outside created bundles.

rdbende commented 1 year ago

Flatpak has many app repositories. The most popular is https://flathub.org. You have to open a pull request to add your app here at https://github.com/flathub/flathub

Moosems commented 1 year ago

I made an issue for Linux distribution but this PR has accomplished goals and is ready to merge.