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

Add Accordion styles based on TitledPane style #30

Closed
dukke opened this issue Jan 1, 2019 · 21 comments
Closed

Add Accordion styles based on TitledPane style #30

dukke opened this issue Jan 1, 2019 · 21 comments

Comments

@dukke
Copy link
Member

dukke commented Jan 1, 2019

Add Accordion styles based on TitledPane style

@MairwunNx
Copy link
Contributor

Good issue, i can you help with it? Or only review your changes? :)

@dukke
Copy link
Member Author

dukke commented Jan 4, 2019

Sure thing.

Would you consider submitting PR. Do you want a few pointers on how to do it?

@MairwunNx
Copy link
Contributor

@dukke I do not know how to do PR. Yes :)

@dukke
Copy link
Member Author

dukke commented Jan 5, 2019

@MairwunNx Sent you an email with some instructions. Cheers 👍

@MairwunNx
Copy link
Contributor

@dukke Thanks bro! I see you instructions, if i do changes with accordion, i make pull request .)))

@MairwunNx
Copy link
Contributor

Ahaha, TitledPane style == Accordion style, bro, accordion already done :DD

image

But, one minus, bottom border real bad, i think you can it fix easy, as i'm :D

image

@MairwunNx
Copy link
Contributor

I think, you need just fix bottom border and close it issue, also thanks for PR instructions on future :)
If i want to change colors \ components - i create PR. 👍

@dukke
Copy link
Member Author

dukke commented Jan 6, 2019 via email

@MairwunNx
Copy link
Contributor

@dukke no problem :) 👍

@MairwunNx
Copy link
Contributor

@dukke Hi! How are you? Have you fixed border? :D

@dukke
Copy link
Member Author

dukke commented Jan 9, 2019

Good @MairwunNx , thanks!

Sorry too much work (on my paid job) 😞

Will try to add today.

@dukke
Copy link
Member Author

dukke commented Jan 10, 2019

@MairwunNx Sorry for the delay in fixing this issue. These days have been though, work wise.
I'll push a fix tomorrow for sure.

Thanks.

@MairwunNx
Copy link
Contributor

@dukke Good, no problem :)) Goodluck with fixing bottom border of accordion 👍

@dukke
Copy link
Member Author

dukke commented Jan 11, 2019

Hi Pavel,

Worked a bit more on it today but I still haven't figured out why the last Titled Pane is being cut at the bottom...

@MairwunNx
Copy link
Contributor

@dukke hi, i think i know what caused bottom not correct border, i need style, you already commit changes with pane?

@dukke
Copy link
Member Author

dukke commented Jan 11, 2019

The Titled Pane was already commited a few weeks ago.

I've fixed it. Only want to fix one more detail that has appeared: "the disabled border of the content of TitledPane isn't being the same as the border color of the Titled Pane header.
Hopefully I'll still have time today to make this small fix.

@MairwunNx
Copy link
Contributor

You can see my changes, scroll up, and see as i make it :), maybe it help you.

@dukke dukke closed this as completed in 0019345 Jan 13, 2019
@dukke
Copy link
Member Author

dukke commented Jan 13, 2019

This has been fixed and pushed. 👍

@MairwunNx
Copy link
Contributor

Good Job @dukke ! 👍

What next tasks?

@dukke
Copy link
Member Author

dukke commented Jan 13, 2019

Thanks @MairwunNx !

I haven't really look into that but there are a few things I'd like to do:

  • TreeTableView styles
  • SplitPane?
  • Dialogs?
  • Accent color constants for easily changing to a nice accent color

@MairwunNx
Copy link
Contributor

@dukke Hi! Thanks for // todo, i try make what i can realize, i also try to realize dialogs with supporting dark and light theme, and others your controls. 👍

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

No branches or pull requests

2 participants