Disable Close and TaskSwitcher buttons on Maemo6

RESOLVED FIXED

Status

Fennec Graveyard
General
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: romaxa, Assigned: romaxa)

Tracking

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

7 years ago
Created attachment 503772 [details] [diff] [review]
Disable home, taskswitcher buttons on maemo6

On maemo6 we don't need close and taskswitcher buttons, similar to Android.

Here is the simple patch to disable it
Attachment #503772 - Flags: review?(mark.finkle)
Comment on attachment 503772 [details] [diff] [review]
Disable home, taskswitcher buttons on maemo6

># HG changeset patch
># Parent b2d7a9386babc231f42313199c66f09d539dd70e
>diff --git a/chrome/content/browser.xul b/chrome/content/browser.xul
>--- a/chrome/content/browser.xul
>+++ b/chrome/content/browser.xul
>@@ -212,17 +212,17 @@
>       <!-- Page Area -->
>       <stack class="window-width window-height">
>         <scrollbox id="page-scrollbox" class="window-width window-height">
>           <vbox>
>             <!-- Main Toolbar -->
>             <box id="toolbar-container" class="panel-dark toolbar-height">
>               <box id="toolbar-moveable-container">
>                 <toolbar id="toolbar-main" class="panel-dark window-width">
>-#ifdef MOZ_PLATFORM_MAEMO
>+#if MOZ_PLATFORM_MAEMO != 6

This would trigger for Android, right? You'll need to keep the outer #ifdef MOZ_PLATFORM_MAEMO

>                     </hbox>
>                   </hbox>
> #ifndef ANDROID
>+#if MOZ_PLATFORM_MAEMO != 6
>                   <toolbarbutton id="tool-app-close" class="urlbar-button" command="cmd_close"/>
> #endif
>+#endif

This one should be OK I think
Attachment #503772 - Flags: review?(mark.finkle) → review-
(Assignee)

Comment 2

7 years ago
Created attachment 503921 [details] [diff] [review]
Updated patch
Assignee: nobody → romaxa
Attachment #503772 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #503921 - Flags: review?(mark.finkle)
Attachment #503921 - Flags: review?(mark.finkle) → review+
(Assignee)

Comment 3

7 years ago
http://hg.mozilla.org/mobile-browser/rev/6ad3f7c8ab25
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

7 years ago
Depends on: 638183
You need to log in before you can comment on or make changes to this bug.