-
Notifications
You must be signed in to change notification settings - Fork 728
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
Creating a custom palette #316
Comments
Hey @ismirlia, thanks for reporting. Custom palettes should work, although none of us have actually used that functionality. So, you're welcome to fix the errors you find in that functionality and submit a pull request with your changes. |
@ismirlia thanks for reporting and @ccordoba12 for the reply. I've been working on this, it is not complete, but its near to work properly. I'll push the changes to the develop branch, so if @ismirlia could test and make any other change it is needed, would be very nice. Thanks again |
That's great news! Thanks a lot for working on that @dpizetta. |
Thank you for the replies @ccordoba12 and @dpizetta . I'll have some time to work on this later in the week and I will take a look and let you know how far I get. |
@ismirlia thanks for your help. I pushed the last changes I've made. That function is still not complete yet, but many changes were already made. We need to get the function args and update the new Palette class template. I've tested with the default values and the creation of palette was fixed, however needs this change for custom ones. |
I was just trying to create a custom palette this weekend. Is the work done in the |
I think @dpizetta hasn't pushed it yet. |
Hello,
I was trying to to create a custom color palette for an application I work with, but I saw that there is a currently an intentional error raised in scss.py
I just wanted to confirm that this functionality has been disabled for the time being? I apologize if this is duplicate, but no one answered #292 and #301 seemed to imply that that was the case, plus they were both close to a year old. Thank you for your time.
The text was updated successfully, but these errors were encountered: