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

manager: Added styles to operation list #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Rushilwiz
Copy link

The plain list was a bit boring, so I added some simple styles to each list item.
screenshot

Styled the operation actions nicer.
@ghost
Copy link

ghost commented Sep 9, 2020

This takes up too much space on the page, and I'm already concerned about the long lists that appear sometimes. Can you remove some of the padding?

Also added some negative margin, and checked it on screen sizes.
@Rushilwiz
Copy link
Author

Rushilwiz commented Sep 9, 2020

Sounds good.

screenshot

@ghost
Copy link

ghost commented Sep 9, 2020

I'm still concerned about padding. Sometimes when a site breaks, the way to fix it is to a "fix site" operation on the site, and that has 7 or 8 actions (depending on whether the site has a database) as part of it. That's a lot of space on the page already, and this would extend it even more.

@Rushilwiz
Copy link
Author

I'm still concerned about padding. Sometimes when a site breaks, the way to fix it is to a "fix site" operation on the site, and that has 7 or 8 actions (depending on whether the site has a database) as part of it. That's a lot of space on the page already, and this would extend it even more.

Yeah, I can see it now, let me to move it down to the bottom, and play with the layout a bit.

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.

1 participant