MiguelCatalan/MaterialSearchView

Bug: reveal animation doesn't always start from the search menu item

Open

#57 ouverte le 20 janv. 2016

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)Java (627 forks)batch import
enhancementhelp wantedon hold

Métriques du dépôt

Stars
 (3 821 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

modify the sample to have more action items on the right of the search menu item, and then click the search menu item. The reveal animation still starts from the most right area, instead of from the search menu item.

Here's an example of the modification that's needed to do it: menu_main.xml

<menu
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    tools:context=".MainActivity">
    <item
        android:id="@+id/action_search"
        android:icon="@drawable/ic_action_action_search"
        android:title="@string/abc_search_hint"
        app:showAsAction="always"/>
    <item
        android:icon="@android:drawable/ic_menu_edit"
        android:title="edit"
        app:showAsAction="always"/>
</menu>

Please fix this.

Guide contributeur

Direction de recherche
Explorez le calcul du point de départ de l'animation dans le code MaterialSearchView. L'animation devrait démarrer à partir de la position de l'élément de menu de recherche, mais elle commence actuellement dans la zone la plus à droite lorsqu'il y a plusieurs éléments d'action. Tracez le code qui détermine l'emplacement de révélation et ajustez le pour obtenir les coordonnées correctes de la vue.
Stack technique
javaandroid
Domaine
mobile
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaAndroid developmentBasic understanding of animations
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60