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

[WIP] Card alignment #126

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

Conversation

newhinton
Copy link

This implements a 'layout' setting, which allows the user to left- right- or centeralign the cards.

This was asked for in #51

Todo: Fix to many cards overflowing the screen when using center

@newhinton newhinton changed the title Card alignment [WIP] Card alignment Apr 15, 2020
@rmountjoy92
Copy link
Owner

see #127 (comment)

@newhinton
Copy link
Author

I see why you dont want to have smaller (or bigger) cards, but why is the alignment of the cards a problem? Generally if they move to the right side or the center should not affect the content of the cards as they are unaffected by the alignment on the mainpage

@rmountjoy92 rmountjoy92 reopened this Apr 20, 2020
@rmountjoy92
Copy link
Owner

Whoops, sorry about that, I lumped these two together. I'll see if I can get this change looking good on 0.6 👍

@newhinton
Copy link
Author

@rmountjoy92 Quick question: i have experimented with coloured cards, is that something i should create a pr for or is that something that is already done or not wanted?

@rmountjoy92
Copy link
Owner

rmountjoy92 commented Apr 22, 2020

@rmountjoy92 Quick question: i have experimented with coloured cards, is that something i should create a pr for or is that something that is already done or not wanted?

I think that's a great idea! You should definitely use the css variables/classes in this file:
https://github.com/rmountjoy92/DashMachine/blob/master/dashmachine/static/css/global/dashmachine-theme.css
So you can access the theme colors.
(feel free to add more variables/classes that you need)

@newhinton
Copy link
Author

this is also a good idea, but actually i thought of card-scoped colors, so that you can make nextcloud the color of your instance, and github a deeper black, and your mailclient green, you get the idea ^^

@newhinton
Copy link
Author

by the way, do you have a remote repo where you merge the stuf, or how does your workflow work? You seem to only close pr's but not actually merge them :D

@rmountjoy92
Copy link
Owner

So, here's how I've been doing it (working on putting this in the docs):

Master is stable branch, it is only changed when a major version from the develop branch is merged into it.

Right now I'm working on 0.6 (the next major version) and most of the changes are local on my computer (keep forgetting to push). If a pr is put on the develop branch, most of time I'll just merge it in (sometimes I see what pr is trying to accomplish, and just do it myself). If it's made on the master branch I've just been making the changes on my copy and closing the pr.

@newhinton
Copy link
Author

#151

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