Skip to content

Change tables styles for Saved Searches - #125

Merged
ImTheKai merged 1 commit into
hackorum-dev:mainfrom
moozzi:saved_searches_table
Aug 19, 2026
Merged

Change tables styles for Saved Searches#125
ImTheKai merged 1 commit into
hackorum-dev:mainfrom
moozzi:saved_searches_table

Conversation

@moozzi

@moozzi moozzi commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

I set columns widths to fixed 50% and 40% for name and query.

Before:
Screenshot 2026-08-19 at 08 45 37
After:
Screenshot 2026-08-19 at 08 46 06

@ImTheKai

Copy link
Copy Markdown
Collaborator

I already looked at it this morning, and now gave it some time and looked at it again. I still cannot say I really like it. I get your intention here, but now the left side part looks so empty and the buttons are not aligned anymore and are left and right across the site. I don't have a good idea top of mind, just my 2 cents.

@moozzi

moozzi commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

and the buttons are not aligned anymore and are left and right across the site.

What do you mean by that?

@moozzi

moozzi commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

I can agree that these new tables are no the best, but imo it's better than the current tables where each column has different width.

We could probably think about a way of removing the empty spaces, but I'm not sure if it's worth waisting a lot of time on it.

@ImTheKai

Copy link
Copy Markdown
Collaborator

and the buttons are not aligned anymore and are left and right across the site.

What do you mean by that?

image

@moozzi

moozzi commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Yeah, but it's a screenshot from before the changes were made, so I think having aligned buttons is better

@ImTheKai

Copy link
Copy Markdown
Collaborator

Yeah, but it's a screenshot from before the changes were made, so I think having aligned buttons is better

Oh lord, I confused before and after, totally sorry. Yes of course this looks way better. Lemme merge

@ImTheKai
ImTheKai merged commit 8a3eb9d into hackorum-dev:main Aug 19, 2026
4 of 5 checks passed
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.

2 participants