Closed Bug 1038924 Opened 6 years ago Closed 6 years ago

Search activity: Review class names and packaging

Categories

(Firefox for Android Graveyard :: Search Activity, defect)

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: eedens, Assigned: Margaret)

References

Details

After the dust settles from the UI/UX work week, we should do a pass over the packaging and naming.

From bug 1038775: I find the terms "presearch" and "postsearch" a bit confusing. Do we even need states here? I would like to think about ways we can rename the fragments to be more intuitive. Perhaps something like InitialFragment and ResultsFragment? I'm having a tough time coming up with a good name for the first one.

Also, SearchFragment could probably be renamed to AutoCompleteFragment, since the whole activity is about searching, so using the term in there can be confusing.
(In reply to Eric Edens [:eedens] from comment #0)
> After the dust settles from the UI/UX work week, we should do a pass over
> the packaging and naming.
> 
> From bug 1038775: I find the terms "presearch" and "postsearch" a bit
> confusing. Do we even need states here? I would like to think about ways we
> can rename the fragments to be more intuitive. Perhaps something like
> InitialFragment and ResultsFragment? I'm having a tough time coming up with
> a good name for the first one.

I don't like InitialFragment, 'cuz it doesn't say what it does.  It'll probably show recent searches, or suggested contents, or your location, or ...

SearchResultsFragment seems reasonable.

> Also, SearchFragment could probably be renamed to AutoCompleteFragment,
> since the whole activity is about searching, so using the term in there can
> be confusing.

Fine by me.
(In reply to Nick Alexander :nalexander from comment #1)
> (In reply to Eric Edens [:eedens] from comment #0)
> > After the dust settles from the UI/UX work week, we should do a pass over
> > the packaging and naming.
> > 
> > From bug 1038775: I find the terms "presearch" and "postsearch" a bit
> > confusing. Do we even need states here? I would like to think about ways we
> > can rename the fragments to be more intuitive. Perhaps something like
> > InitialFragment and ResultsFragment? I'm having a tough time coming up with
> > a good name for the first one.
> 
> I don't like InitialFragment, 'cuz it doesn't say what it does.  It'll
> probably show recent searches, or suggested contents, or your location, or
> ...
> 
> SearchResultsFragment seems reasonable.

Well, in the future it may show more. We've been throwing around the term "dashboard" here, but that might imply more than what this fragment is actually doing. However, the things we've been talking about are all related to your search history, so maybe SearchHistoryFragment would work as a name.
We're iterating on this as we go. Things are fine for now, we can change them in the bugs where we need them to be changed.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.