ionic-team / ionic-framework

A powerful cross-platform UI toolkit for building native-quality iOS, Android, and Progressive Web Apps with HTML, CSS, and JavaScript.
https://ionicframework.com
MIT License
51.09k stars 13.51k forks source link

bug: inconsistency in the style of highlighted dates #29428

Closed GerardoPrototype closed 6 months ago

GerardoPrototype commented 6 months ago

Prerequisites

Ionic Framework Version

v7.x

Current Behavior

When you add marked dates to the calendar it displays the style correctly. but when you select that date it loses the style

Captura de pantalla 2024-04-30 a la(s) 5 13 17 p m

Expected Behavior

the desired behavior would be. When selecting a marked date, it does not delete its style.

Captura de pantalla 2024-04-30 a la(s) 5 04 02 p m

This design where I can see a highlighted date and with the active style is currently impossible

Steps to Reproduce

create an ion-date component with highlighted dates. Select a date and see how your style is eliminated

Code Reproduction URL

https://ionicframework.com/docs/api/datetime#using-array

Ionic Info

Ionic:

Ionic CLI : 7.2.0 (/Users/gerardo/.nvm/versions/node/v18.18.0/lib/node_modules/@ionic/cli) Ionic Framework : @ionic/angular 7.8.4 @angular-devkit/build-angular : 17.1.3 @angular-devkit/schematics : 17.1.3 @angular/cli : 17.1.3 @ionic/angular-toolkit : 10.1.1

Capacitor:

Capacitor CLI : 5.7.4 @capacitor/android : 5.7.4 @capacitor/core : 5.7.4 @capacitor/ios : 5.7.4

Utility:

cordova-res : not installed globally native-run : 2.0.1

System:

NodeJS : v18.18.0 (/Users/gerardo/.nvm/versions/node/v18.18.0/bin/node) npm : 10.1.0 OS : macOS Unknown

Additional Information

No response

liamdebeasi commented 6 months ago

Thanks for the issue. I am going to close this as this is not a bug in Ionic Framework. As explained on the Datetime documentation, the style of the selected date(s) will always override custom highlights. This was done to ensure that users always know which dates are selected.

ionitron-bot[bot] commented 5 months ago

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.