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.01k stars 13.51k forks source link

bug: ion-menu-button doesn't change font-size in md mode #27189

Closed beyondraymond closed 1 year ago

beyondraymond commented 1 year ago

Prerequisites

Ionic Framework Version

v6.x

Current Behavior

I changed the font-size of my ion-menu-button (hamburger button) using the following CSS code.

ion-menu-button { font-size: 31px !important; }

The above code seems to work on my app in ios mode but that was not the case when I tried running the app on md mode(Android). The hamburger button is still small, even the CSS code doesn't have an impact.

Attached screenshot below is from the md mode of my Ionic app. Notice how the hamburger button is small. image

Expected Behavior

Both the ios and md mode should have the same font-size for the ion-menu-button after adding the abovementioned CSS code. Attached screenshot below is from the ios mode of my Ionic app.

image

Steps to Reproduce

  1. On the .SCSS file of your app, enter the following code to make the logo appear larger. ion-menu-button { font-size: 31px !important; }
  2. Run on your app on ios mode. Check if the hamburger button has changed its font-size.
  3. Run on your app on md mode. The same font-size should be observed in md mode.

Code Reproduction URL

No response

Ionic Info

Ionic:

Ionic CLI : 6.20.6 (C:\Users\Raymond\AppData\Roaming\npm\node_modules\@ionic\cli) Ionic Framework : @ionic/angular 6.4.1 @angular-devkit/build-angular : 15.0.4 @angular-devkit/schematics : 15.0.4 @angular/cli : 15.0.4 @ionic/angular-toolkit : 6.1.0

Capacitor:

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

Utility:

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

System:

NodeJS : v18.12.0 (C:\Program Files\nodejs\node.exe) npm : 8.19.2 OS : Windows 10

Additional Information

No response

liamdebeasi commented 1 year ago

Thanks for the report. Can you reproduce this in an Ionic starter app and provide a link to the repo? I cannot reproduce this on my end: https://codepen.io/liamdebeasi/pen/JjmGNXx

ionitron-bot[bot] commented 1 year ago

Thanks for the issue! This issue is being closed due to the lack of a reply. 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.

Thank you for using Ionic!