Skip to content

Style - #119

Merged
twynb merged 7 commits into
finalise-docsfrom
style
Sep 24, 2023
Merged

Style#119
twynb merged 7 commits into
finalise-docsfrom
style

Conversation

@phtngyn

@phtngyn phtngyn commented Sep 24, 2023

Copy link
Copy Markdown
Collaborator

No description provided.

@phtngyn phtngyn self-assigned this Sep 24, 2023
@phtngyn phtngyn mentioned this pull request Sep 24, 2023
@twynb
twynb self-requested a review September 24, 2023 14:42

@twynb twynb left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

attempting to build throws a build error, because PyInstaller doesn't accept svg files as app icons. Please re-add logo.ico and use it in build.spec.

@phtngyn

phtngyn commented Sep 24, 2023

Copy link
Copy Markdown
Collaborator Author

attempting to build throws a build error, because PyInstaller doesn't accept svg files as app icons. Please re-add logo.ico and use it in build.spec.

resolved

@phtngyn
phtngyn requested a review from twynb September 24, 2023 15:08

@twynb twynb left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if I'm a fan of this design - the way it is now, the input fields/switches are way too far away from the labels, which makes them feel disconnected. I think having them closer together would be better.

@phtngyn

phtngyn commented Sep 24, 2023

Copy link
Copy Markdown
Collaborator Author

When you increase window width a little then max width will be set again to 80%. What do you think?
When not, I will set back to lg:max-w-80%

@twynb

twynb commented Sep 24, 2023

Copy link
Copy Markdown
Owner

That behavior doesn't happen on my end - when i widen the window, the max width grows a little, but nowhere near where it was before

@phtngyn

phtngyn commented Sep 24, 2023

Copy link
Copy Markdown
Collaborator Author

That behavior doesn't happen on my end - when i widen the window, the max width grows a little, but nowhere near where it was before

have reverted

@phtngyn

phtngyn commented Sep 24, 2023

Copy link
Copy Markdown
Collaborator Author

@ChrisItisdud can you also check whether icon's quality in system tray after build is better or not. I cant check because in linux there is no icon

@twynb

twynb commented Sep 24, 2023

Copy link
Copy Markdown
Owner

i can't see the change being reverted, to me building the app and running the built app looks like this:

grafik

maybe those xl/lg rules don't work properly when running in an app like this?

also i never bothered to check the tray icon quality, it does look pretty low res though

@twynb twynb left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks better now - not perfect but i think further changes can be saved for another issue after publishing

@twynb
twynb merged commit 0f9ab5f into finalise-docs Sep 24, 2023
twynb added a commit that referenced this pull request Sep 24, 2023
* finalise docs

* delete no longer necessary files

* Style (#119)

* Fix Doc tags

* style: update

* revert: change

* add better logo.ico

* revert: change in settings

* style

---------

Co-authored-by: 4lex0017 <alex.kern@live.de>
Co-authored-by: Christina / Itisdud / twyn!b <92167981+ChrisItisdud@users.noreply.github.com>

* finalise docs

* chore: remove d.ts

---------

Co-authored-by: phat.nguyen <89086602+chubetho@users.noreply.github.com>
Co-authored-by: 4lex0017 <alex.kern@live.de>
Co-authored-by: phatnguyen <phatnguyen@gmx.de>
@phtngyn
phtngyn deleted the style branch September 25, 2023 14:39
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

Successfully merging this pull request may close these issues.

3 participants