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

Some minor waybar fixes and enhancements #1469

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

its-19818942118
Copy link

@its-19818942118 its-19818942118 commented May 10, 2024

Pull Request

Description

some minor fixes of waybar alignments issues. added fonts but didnt know how to make so the scripts read the fonts. waybar works fine. added seconds hand on the clock.

Please read these instructions and remove unnecessary text.

Type of change

Please put an x in the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (non-breaking change; modified files are limited to the documentations)
  • Technical debt (a code change that does not fix a bug or add a feature but makes something clearer for devs)
  • Other (provide details below)

Checklist

Please put an x in the boxes that apply:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My commit message follows the commit guidelines.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added a changelog entry.
  • I have added necessary comments/documentation to my code.
  • I have added tests to cover my changes.
  • I have tested my code locally and it works as expected.
  • All new and existing tests passed.

Screenshots

(if appropriate)

Additional context

Add any other context about the problem here.

@its-19818942118
Copy link
Author

@abenezerw
Copy link
Contributor

image
looks fine here.

Copy link
Contributor

@abenezerw abenezerw left a comment

Choose a reason for hiding this comment

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

Checked the files out on my pc. they work fine on most of the styles of the waybar, on some the whole thing becomes zoomed in.

@rubiin
Copy link
Contributor

rubiin commented May 13, 2024

Checked the files out on my pc. they work fine on most of the styles of the waybar, on some the whole thing becomes zoomed in.

can you also share the zoomed in screenshots. Will help in reviewing pr

@kRHYME7
Copy link
Collaborator

kRHYME7 commented May 13, 2024

default:
image
image
image

PR
image
image
image

TBH, I really like it! I did not check the files changed though, but please be sure not to hard code some values if possible, in case.

Overall, this is neat! Will further test this.

Spacing issues:
image

@abenezerw
Copy link
Contributor

Checked the files out on my pc. they work fine on most of the styles of the waybar, on some the whole thing becomes zoomed in.

can you also share the zoomed in screenshots. Will help in reviewing pr

image

@abenezerw
Copy link
Contributor

  1. By the way im using a lenovo legion laptop and it has a feature where you can set the battery to be stuck around 60% and use it while charging to prevent long term battery damage, i always use it charging and was wondering if the script has any parts i could alter to swap to plugged in icon in this state.
  2. A suggestion for the network/wifi icons. why not combine them i personally like the three dotted wifi icon. but the other one where it shows network when hovered on seems to be unused. maybe combine them to have more space?

@kRHYME7
Copy link
Collaborator

kRHYME7 commented May 13, 2024

@abenezerw

re:1. This is handled by the battery module of waybar itself, so the display is based on the default waybar program. This is not handled by any user scripts

re:2. we can't the other one is a system tray Icon other is a module.(if someone wants to make custom script to launch rofi and works like how nm-applet does, it would be nice. )

@abenezerw
Copy link
Contributor

cool, ill go try the swaync pr next.

@kRHYME7
Copy link
Collaborator

kRHYME7 commented May 13, 2024

One more note to anyone tetsing a PR and preserving their current configs

Hyde backup all backs up your config (name it to something you remember)

Hyde --dev then select pull request

Hyde restore Config to populate the Configs
repeat

to revert back to your back up Hyde backup revert select the backup.

@its-19818942118
Copy link
Author

its-19818942118 commented May 13, 2024

default: image image image

PR image image image

TBH, I really like it! I did not check the files changed though, but please be sure not to hard code some values if possible, in case.

Overall, this is neat! Will further test this.

Spacing issues: image

tell me the moddules which have spacing issues. none of this is tested on local cuz my waybar is broken. but yeah if you let me know i can fix it. i can also make more custom layouts.

edit: this seems to be like screen scaling. cuz i have a small screen. so it was perfect on my side. but i think i need to add mono spaced fonts.

@abenezerw
Copy link
Contributor

default: image image image
PR image image image
TBH, I really like it! I did not check the files changed though, but please be sure not to hard code some values if possible, in case.
Overall, this is neat! Will further test this.
Spacing issues: image

tell me the moddules which have spacing issues. none of this is tested on local cuz my waybar is broken. but yeah if you let me know i can fix it. i can also make more custom layouts.

edit: this seems to be like screen scaling. cuz i have a small screen. so it was perfect on my side. but i think i need to add mono spaced fonts.

i agree with the screen scaling cause the modules dont collide on my end too

@rubiin
Copy link
Contributor

rubiin commented May 13, 2024

default: image image image
PR image image image
TBH, I really like it! I did not check the files changed though, but please be sure not to hard code some values if possible, in case.
Overall, this is neat! Will further test this.
Spacing issues: image

tell me the moddules which have spacing issues. none of this is tested on local cuz my waybar is broken. but yeah if you let me know i can fix it. i can also make more custom layouts.

edit: this seems to be like screen scaling. cuz i have a small screen. so it was perfect on my side. but i think i need to add mono spaced fonts.

Will also give it a test when I am free

@its-19818942118
Copy link
Author

Checked the files out on my pc. they work fine on most of the styles of the waybar, on some the whole thing becomes zoomed in.

can you also share the zoomed in screenshots. Will help in reviewing pr

image

this is because the icon size is in percentage. depending on the space and height it will adjust. i made it so that it isnt hardcoded. i guess need to fix it. but my waybar just broke so i couldnt test any of this

@its-19818942118 its-19818942118 marked this pull request as draft May 13, 2024 14:49
@abenezerw
Copy link
Contributor

abenezerw commented May 13, 2024

Whats the edit?
@its-19818942118

@its-19818942118
Copy link
Author

default: image image image
PR image image image
TBH, I really like it! I did not check the files changed though, but please be sure not to hard code some values if possible, in case.
Overall, this is neat! Will further test this.
Spacing issues: image

tell me the moddules which have spacing issues. none of this is tested on local cuz my waybar is broken. but yeah if you let me know i can fix it. i can also make more custom layouts.
edit: this seems to be like screen scaling. cuz i have a small screen. so it was perfect on my side. but i think i need to add mono spaced fonts.

Will also give it a test when I am free

sure and will be adding more functionality over all

@its-19818942118
Copy link
Author

Whats the edit? @its-19818942118

i shared lol

@its-19818942118
Copy link
Author

also someone pls test pulse audio cuz my system doesnt have audio. so nothing works. no audio cuz my drivers dont work idk why. but yeah let me if you find something

@abenezerw
Copy link
Contributor

image
There's this text on hover.

@its-19818942118
Copy link
Author

ok i just tested this but mono spaced fonts make it look janky and unprofessional. so unfortunately i cant implement a mono spaced font it ruins it. check the before and after you guys. trust me it makes the older version look shit

@its-19818942118
Copy link
Author

its-19818942118 commented May 13, 2024

also @kRHYME7 i found the issue its seems to be the network icon that is interfering and making it collide

edit: its the backlight icon and the spacing of it. cuz if you look closely you will see that there is too much space. like 2 spaces. so yeah. let me see what i can do

@abenezerw
Copy link
Contributor

abenezerw commented May 13, 2024

ok i just tested this but mono spaced fonts make it look janky and unprofessional. so unfortunately i cant implement a mono spaced font it ruins it. check the before and after you guys. trust me it makes the older version look shit

Haha, did you fix the *span issue

@its-19818942118
Copy link
Author

ok i just tested this but mono spaced fonts make it look janky and unprofessional. so unfortunately i cant implement a mono spaced font it ruins it. check the before and after you guys. trust me it makes the older version look shit

Haha, did you fix the *span issue

@abenezerw which one

@abenezerw
Copy link
Contributor

ok i just tested this but mono spaced fonts make it look janky and unprofessional. so unfortunately i cant implement a mono spaced font it ruins it. check the before and after you guys. trust me it makes the older version look shit

Haha, did you fix the *span issue

@abenezerw which one

Hover over brightness, audio, mic or updates. It shows extra text. See the screenshot I left above.

@its-19818942118
Copy link
Author

ok i just tested this but mono spaced fonts make it look janky and unprofessional. so unfortunately i cant implement a mono spaced font it ruins it. check the before and after you guys. trust me it makes the older version look shit

Haha, did you fix the *span issue

@abenezerw dont have it on my end. i dont get any tooltip lol

@its-19818942118
Copy link
Author

ok i just tested this but mono spaced fonts make it look janky and unprofessional. so unfortunately i cant implement a mono spaced font it ruins it. check the before and after you guys. trust me it makes the older version look shit

Haha, did you fix the *span issue

@abenezerw broo i didnt get this issue on my end lol

@its-19818942118
Copy link
Author

aight guys created a dev server just for feedback on pr and new stuff and dev discussion. cuz the HyDE server just makes it cluttered and people feel lost. plus github isnt really intuitive for chatting. if you want to you can join the server.
https://discord.gg/FYJQT9DReV

@its-19818942118
Copy link
Author

Ok the tooltip thing is fixed substitution with hard coded icons cuz if not it still prints out the guts that fixes the icons. Had not choice 😕😞

@its-19818942118 its-19818942118 marked this pull request as ready for review May 25, 2024 14:54
@its-19818942118
Copy link
Author

@kRHYME7 @prasanthrangan ready to go. pls approve this. i need more feedback on this.

Copy link
Collaborator

@kRHYME7 kRHYME7 left a comment

Choose a reason for hiding this comment

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

This "minor" is not minor to me. Well, looks good though! 🫰

@its-19818942118
Copy link
Author

This "minor" is not minor to me. Well, looks good though! 🫰

what compared to oothers works mine is minimal af. people have good shit i am just tweaking a bit of the waybar to make the ui look nice. lol @kRHYME7

@kRHYME7
Copy link
Collaborator

kRHYME7 commented Jun 17, 2024

Hard-coded colors too.

@its-19818942118
Copy link
Author

Well yeah colors need to be hardcoded or it looks messed up 😅

@its-19818942118
Copy link
Author

@prasanthrangan

@its-19818942118
Copy link
Author

@rishav12s

@dieBakterie
Copy link
Contributor

The right click entry for the audio selector is missing in your pulseaudio.jsonc

@its-19818942118
Copy link
Author

The right click entry for the audio selector is missing in your pulseaudio.jsonc

i added it. i am sure i did

Copy link
Contributor

@abenezerw abenezerw left a comment

Choose a reason for hiding this comment

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

Finally got to trying this out again, still looks okay. but i guess this counts as big changes so its not merged maybe.

@its-19818942118
Copy link
Author

@abenezerw 😭 io will just close this

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.

6 participants