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

scrollbars not need but displayed, e.g. on ActionSheet, on Windows Chrome #13966

Closed
ghost opened this issue Feb 6, 2018 · 5 comments
Closed
Labels
ionitron: v3 moves the issue to the ionic-v3 repository

Comments

@ghost
Copy link

ghost commented Feb 6, 2018

Ionic version:
[ x] 3.9.2

I'm submitting a ...
[ x] bug report

Current behavior:
ActionSheet displays scrollbars.

Expected behavior:
ActionSheet does not display scrollbars.

Steps to reproduce:

ionic start myApp tabs

add button to home page, with presentActionSheet() handler

ionic serve

in Chrome Version 64.0.3282.140 (Official Build) (64-bit)
scrollbars are displayed:

scrollbars-actionsheet

Related code:

  public presentActionSheet() {
    let actionSheet = this.actionSheetCtrl.create({
      title: 'Modify your album',
      buttons: [
        {
          text: 'Destructive',
          role: 'destructive',
          handler: () => {
            console.log('Destructive clicked');
          }
        },
        {
          text: 'Archive',
          handler: () => {
            console.log('Archive clicked');
          }
        },
        {
          text: 'Cancel',
          role: 'cancel',
          handler: () => {
            console.log('Cancel clicked');
          }
        }
      ]
    });

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):

cli packages: (C:\Users\uzimskacel\AppData\Roaming\npm\node_modules)

    @ionic/cli-utils  : 1.19.0
    ionic (Ionic CLI) : 3.19.0

local packages:

    @ionic/app-scripts : 3.1.8
    Ionic Framework    : ionic-angular 3.9.2

System:

    Node : v8.9.1
    npm  : 5.5.1
    OS   : Windows 10

Misc:

    backend : legacy
@ghost
Copy link
Author

ghost commented Feb 6, 2018

Maybe related to #13763?

@StefanRein
Copy link

@miloslavskacel No it is not related.

Just add

.action-sheet-group { overflow: auto; }

To reproduce the issue:

ionic start tabs
IonicModule.forRoot(MyApp, {mode: 'md'})

add to home.html
<p><button ion-button block (click)="presentActionSheet()">presentActionSheet</button></p>

Use above code and actionSheet.present();

The scrollbars are not visible if you are using the emulator (mac and windows).

image

If you are not using the emulator (on windows, or mac with scrollbars always visible) the scrollbars will appear.

Windows:
image

Mac (with options for scrollbars always on):
image

With overflow: auto

image

StefanRein added a commit to StefanRein/ionic that referenced this issue Feb 9, 2018
@ghost
Copy link
Author

ghost commented Feb 9, 2018

Thanks.
The same problem is on class="scroll-content".
There is a vertical scroll bar on the screenshot.

@Ionitron Ionitron added the ionitron: v3 moves the issue to the ionic-v3 repository label Nov 29, 2018
@ionitron-bot
Copy link

ionitron-bot bot commented Nov 29, 2018

This issue has been automatically identified as an Ionic 3 issue. We recently moved Ionic 3 to its own repository. I am moving this issue to the repository for Ionic 3. Please track this issue over there.

If I've made a mistake, and if this issue is still relevant to Ionic 4, please let the Ionic Framework team know!

Thank you for using Ionic!

@ionitron-bot
Copy link

ionitron-bot bot commented Nov 29, 2018

Issue moved to: ionic-team/ionic-v3#403

@ionitron-bot ionitron-bot bot closed this as completed Nov 29, 2018
@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Nov 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ionitron: v3 moves the issue to the ionic-v3 repository
Projects
None yet
Development

No branches or pull requests

2 participants