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

Two buttons in ion-nav-buttons side right on Android 4.4 #930

Closed
dovidkopel opened this issue Mar 27, 2014 · 18 comments
Closed

Two buttons in ion-nav-buttons side right on Android 4.4 #930

dovidkopel opened this issue Mar 27, 2014 · 18 comments

Comments

@dovidkopel
Copy link

With two buttons in my ion-nav-buttons tag with side="right" the second button is not floated properly, and goes goes into the content.

I have the "exact" same app running on the same phone model but it is working. Not sure why this is happening intermittently.

screenshot_2014-03-27-12-40-29

@ajoslin
Copy link
Contributor

ajoslin commented Mar 28, 2014

What browser/version does it not work on?

@dovidkopel
Copy link
Author

This was happening on a Galaxy Note 3...I have two of them, one Verizon and on Sprint. Thee only reason I mention that is because the Sprint one is running Kit Kat....this happened on the one for Kit Kat.

@niahmiah
Copy link

I noticed this too when trying to override the right nav buttons. When I provide a view with 2 buttons, instead of them replacing the 1 button, a new div was created underneath the previous one, causing me to have 3 right buttons in 2 rows, one of which overlapped into the content area.

Using 1.0.0-beta on safari

EDIT: To get around this issue, I had to remove the ion-nav-bar from my ion-pane, and put a specific ion-nav-bar in every view. Not as ideal as how I was updating the nav bar in 0.9.26, but it works for now.

@ajoslin
Copy link
Contributor

ajoslin commented Mar 29, 2014

Ah, I see the problem! Thanks.

Will fix.

On Fri, Mar 28, 2014 at 2:49 PM, niahmiah [email protected] wrote:

I noticed this too when trying to override the right nav buttons. When I
provide a view with 2 buttons, instead of them replacing the 1 button, a
new div was created underneath the previous one, causing me to have 3 right
buttons in 2 rows, one of which overlapped into the content area.

Using 1.0.0-beta on safari

Reply to this email directly or view it on GitHubhttps://github.com//issues/930#issuecomment-38966349
.

@ajoslin
Copy link
Contributor

ajoslin commented Mar 31, 2014

Fixed! From the commit message:

Uses a <span> now for each set buttons in the navbar, instead of a
<div>. This means that the buttons will align correctly instead of
creating new lines for each set of buttons.

@dovidkopel
Copy link
Author

I have this fix and am still having the same result here is the code:

    <ion-view title="Notifications">
        <ion-nav-buttons side="left">
            <button 
                class="button button-icon icon ion-navicon"
                menu-toggle="left">
            </button>
        </ion-nav-buttons>
        <ion-nav-buttons side="right">
            <button
                class="button button-icon icon ion-close-circled"
                ng-click="requestClearNotifications()">
            </button>        
            <button
                class="button button-icon icon ion-alert-circled"
                blink
                blink-class="energized"
                blink-interval="750"
                ng-click="notificationClick()"
                ng-show="hasUnreadNotifications()">
            </button>
        </ion-nav-buttons>  
           .....

I still am NOT experiencing this issue on Android 4.3, but on 4.4.2 I am.

@ajoslin
Copy link
Contributor

ajoslin commented Apr 23, 2014

Hi @kopeltech,

This is actually a different problem, but definitely weird. Only getting this on Android 4.4 too! css, why?

@ajoslin ajoslin reopened this Apr 23, 2014
@vincentjames501
Copy link

+1. Same issue here on 4.4+

@dovidkopel dovidkopel changed the title Two buttons in ion-nav-buttons side right Two buttons in ion-nav-buttons side right on Android 4.4 May 2, 2014
@ajoslin ajoslin closed this as completed May 13, 2014
@hai1xu
Copy link

hai1xu commented Aug 6, 2014

+1, have the same problem with 4.4.2. For both left and right ion-nav-button.

@bwalsh
Copy link

bwalsh commented Aug 18, 2014

+1, have this problem. Happens with Android 4.4.4 (19) . iOS 7 (7.0) is fine. Web (google chrome) is fine.

@mrvamsidhar
Copy link

+1. I am also having this issue in Android kitkat. I tried simulating this issue for some time. And found that, when there are more than one button inside ion-nav-buttons, the width to adjust the buttons inside ion-nav-buttons is falling short.

Lets say, I have two buttons, one is 54px width and other is 45px. the width for these buttons is is exactly 99px - which was not enough for ion-nav-buttons. That is reason it is wrapping to next line. In debugger console, if you try altering width (say 102px). it works perfectly fine.
Hope this might be issue.

@a-katsarov
Copy link

.platform-android .secondary-buttons button,
.platform-android .primary-buttons button {
                                            float: left;
                                            }

That works for me as a temporary solution.

@mhartington
Copy link
Contributor

This should be fixed in the nightly build via eef1d32

@durajski
Copy link

Try to remove all the spaces between tags on ion-nav-buttons. Take a look on this topic:
http://forum.ionicframework.com/t/multiple-navbar-icons-on-android-align-vertically-instead-of-horizontally/3258/4

@castrojr913
Copy link

The same issue with Samsung devices. I tried on Nexus and Huawei phones and It worked correctly.

@ChrisDraper
Copy link

I had the same issue on Sony running Android 4.4 and this solved the issue. Thanks Mike.

@ale1981
Copy link

ale1981 commented Feb 1, 2017

I also had the same issue on 4.4.2, adding comment blocks <!-- --> between the button elements worked for me.

@ionitron-bot
Copy link

ionitron-bot bot commented Sep 4, 2018

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.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests