tuttle-dev / tuttle

🪰 Tuttle - time and money management for freelancers
GNU General Public License v3.0
63 stars 12 forks source link

UI: Sidebar layout #214

Closed clstaudt closed 1 year ago

clstaudt commented 1 year ago

Image

clstaudt commented 1 year ago

@flamesjames Good first issue?

flamesjames commented 1 year ago

@flamesjames Good first issue?

@clstaudt - yes thank you, assigned myself. Do you have a preference of whether you'd like me to work on this or https://github.com/tuttle-dev/tuttle/issues/14 first?

clstaudt commented 1 year ago

@flamesjames No, take your pick. Both are good starting points.

flamesjames commented 1 year ago

@clstaudt - do you mind helping point me in the right direction of where I should be looking? I am currently checking app/home/view.py but having trouble determining what object the navigation control is. I see where it's being created in MainMenuItemsHandler class..

clstaudt commented 1 year ago

@flamesjames It might help you to know that the menus are of type (views.TNavigationMenu).

@vlad-ed-git any other useful hints we can give?

flamesjames commented 1 year ago

@clstaudt - the class helps a lot, thanks Christian. I will take a look in free time on it. Much appreciated, will reach out if taking too much time and stuck.

flamesjames commented 1 year ago

@clstaudt - apologies.. I'm having a lot of trouble determining how to solve this issue.. you can unassign me and give to someone else.. I'm not sure on the correct approach.

clstaudt commented 1 year ago

@flamesjames Sure, no problem. Regarding your difficulties, what were they? Do you feel you lack some basic knowledge (Python, flet/Flutter, ...) or is it something we could fix (better design, documentation, ...)?

flamesjames commented 1 year ago

@clstaudt - I feel it's probably a combination of multiple points you listed.. new to Python and Open Source, not familiar with flet/flutter, and was tough to determine where I had to manipulate to solve the issue so not sure if better documentation would help..

vlad-ed-git commented 1 year ago

@clstaudt @flamesjames fixed in #223