If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Remove existing about:home view

RESOLVED FIXED in Firefox 26

Status

()

Firefox for Android
Theme and Visual Design
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: sriram, Assigned: sriram)

Tracking

unspecified
Firefox 26
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-fig)

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
The long vertical about:home should go. Now that ViewPager is added, we can remove this single page and start adding individual pages.

(Sorry :bnicholson, but this should go :( )
(Assignee)

Updated

4 years ago
Blocks: 862793
(Assignee)

Updated

4 years ago
Assignee: nobody → sriram
(Assignee)

Comment 1

4 years ago
Created attachment 751147 [details] [diff] [review]
Patch

This hides the widget/AboutHome from the HomePager. Later when we add individual pages, we can add the update() functionalities. The background is all white now.
Attachment #751147 - Flags: review?(bnicholson)

Comment 2

4 years ago
Comment on attachment 751147 [details] [diff] [review]
Patch

Review of attachment 751147 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/base/home/HomePager.java
@@ +4,5 @@
>   * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
>  
>  package org.mozilla.gecko.home;
>  
> +import org.mozilla.gecko.widget.AboutHome;

Drive-by: I think we can get rid of this AboutHome class altogether, since we're not really using most of it anymore. The parts of it we do need (like UpdateFlags) could move into HomePager.

We could also kill off all the views that are in it. Maybe we want to keep them around as we transition some of their contents into the new about:home, but all that code still exists on mozilla-central, so we always have that as a reference.

I suppose we also need to kill off all the AwesomeBar related stuff at some point, too... so many things are going to go away :)
Comment on attachment 751147 [details] [diff] [review]
Patch

Review of attachment 751147 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/base/home/HomePager.java
@@ +4,5 @@
>   * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
>  
>  package org.mozilla.gecko.home;
>  
> +import org.mozilla.gecko.widget.AboutHome;

I agree that we should kill off AboutHome, and we should probably move/delete all abouthome/awesomebar classes ASAP. If we leave these files as they are, they'll silently be updated when merging from m-c. We want to avoid that since there may be incoming fixes that we end up dropping when these files do eventually get deleted. Since you've gotten rid of the AboutHome dependencies, can you file a follow-up to remove all AboutHome/AwesomeBar-related code?
Attachment #751147 - Flags: review?(bnicholson) → review+

Comment 4

4 years ago
(In reply to Brian Nicholson (:bnicholson) from comment #3)
> Comment on attachment 751147 [details] [diff] [review]
> Patch
> 
> Review of attachment 751147 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: mobile/android/base/home/HomePager.java
> @@ +4,5 @@
> >   * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
> >  
> >  package org.mozilla.gecko.home;
> >  
> > +import org.mozilla.gecko.widget.AboutHome;
> 
> I agree that we should kill off AboutHome, and we should probably
> move/delete all abouthome/awesomebar classes ASAP. If we leave these files
> as they are, they'll silently be updated when merging from m-c. We want to
> avoid that since there may be incoming fixes that we end up dropping when
> these files do eventually get deleted. Since you've gotten rid of the
> AboutHome dependencies, can you file a follow-up to remove all
> AboutHome/AwesomeBar-related code?

I filed bug 876712 for killing off the AwesomeBar code, but we could expand it's scope to kill off the AboutHome stuff, too.
(Assignee)

Comment 5

4 years ago
http://hg.mozilla.org/projects/fig/rev/762d1865d454
(Assignee)

Updated

4 years ago
Whiteboard: fixed-fig

Comment 6

4 years ago
https://hg.mozilla.org/mozilla-central/rev/762d1865d454
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
You need to log in before you can comment on or make changes to this bug.