Giter Club home page Giter Club logo

Comments (21)

dukke avatar dukke commented on July 18, 2024 1

Yes that's right.

I still think the gray color makes it feel somewhat like the button is disabled.

from jfxtras-styles.

dukke avatar dukke commented on July 18, 2024

Thanks, good issue

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

@dukke no problem!

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

@dukke new bugs with date-picker:

Focus traversable button need to disable, because after pressing and lost hover, causes not great effect.

image

image

And it :D

image

from jfxtras-styles.

dukke avatar dukke commented on July 18, 2024

Ok thanks for reporting that.

Could you please, possibly, file another issue for that bug?

Regarding the issue itself I'm not quite understanding what's it about.

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

@dukke hi! I fixed not cool bug with large menu, but: it was intended what not editable date-picker more than editable? (2 row it - not editable, 1 - editable, 3 - text fields)

image

from jfxtras-styles.

dukke avatar dukke commented on July 18, 2024

Hmm. I still don't understand the issue, sorry.

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

@dukke i fixed issue with wrong font size in context-menu for date-picker - understand?)

bug what need to fix:

fix wrong dark theme for date-picker
fix wrong size for date-picker with disabled editable prop
fix wrong font size in context menu in date-picker
fix wrong color selecting today day and selected new day
fix wrong colors of selecting days and day cells

currently i try fix all problems with date-picker.

from jfxtras-styles.

dukke avatar dukke commented on July 18, 2024

Oh yes, I see it now. Wrong text size of Date-Picker context menu. Nice 👍

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

image

fixed different height when disabled editable prop.

from jfxtras-styles.

dukke avatar dukke commented on July 18, 2024

What's this last issue you mention?

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

@dukke
fix wrong size for date-picker with disabled editable prop.
#34 (comment)

What's this last issue you mention?

from jfxtras-styles.

dukke avatar dukke commented on July 18, 2024

is that last issue the wrong font size of the context menu of Date Picker?

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

@dukke not, wrong font size already fixed, i mean wrong size when disabled editable property :)

from jfxtras-styles.

dukke avatar dukke commented on July 18, 2024

Sorry... still not sure. Maybe if you tell me the steps to reproduce?

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

I already done and fix dark theme for date-picker, i think it fully done.

Before:

image

image

image

After:

image

image

image

image

image

image

from jfxtras-styles.

dukke avatar dukke commented on July 18, 2024

👍
Make the calendar icon color also white when the DatePicker is not focused.

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

@dukke I can't do that, this is a direct violation of UX, the user will think that he is focused on the element, when he is not focused on it, if you want, I can do it, but this is not correct

It will also be too contrasted, and then the dark theme may become worse light due to the high contrast and sharpness.

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

Resolved in #45

from jfxtras-styles.

dukke avatar dukke commented on July 18, 2024

The Fluent Design implementation of the Date Picker with a dark style has the Calendar icon white when the calendar is not focused.

image

If it is gray I think it is not totally obvious that you can interact with the calendar icon button.

from jfxtras-styles.

MairwunNx avatar MairwunNx commented on July 18, 2024

In fluent design it make as button with icon :), we have as text-field and combobox :)

needed gray because text-field can be focus-able, bbut if you need write, you can edit color before merging branch

from jfxtras-styles.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.