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

#4643: Home screen: replacing the search icon by a filter icon in the menu #5253

Merged
merged 2 commits into from
Feb 17, 2022

Conversation

mnaturel
Copy link
Contributor

@mnaturel mnaturel commented Feb 17, 2022

Type of change

  • Feature
  • Bugfix
  • Technical
  • Other : Enhancement

Content

Replacing the search icon in the home menu by a filter icon to be more consistent.

Motivation and context

See #4643

Screenshots / GIFs

4643-home-menu-light4643-home-menu-dark

Tests

  • Go to home screen
  • Check a filter icon is displayed in the top right menu
  • Click on the filter icon
  • Check the filter screen is opened

Tested devices

  • Physical
  • Emulator
  • OS version(s): Android 11

Checklist

@mnaturel mnaturel added the Z-WTF WTF moment: High Impact, Low Effort label Feb 17, 2022
changelog.d/4643.misc Outdated Show resolved Hide resolved
@mnaturel mnaturel changed the title #4643: Replacing the search icon by a filter icon #4643: Home screen: replacing the search icon by a filter icon in the menu Feb 17, 2022
@mnaturel mnaturel enabled auto-merge February 17, 2022 10:27
@github-actions
Copy link

Unit Test Results

  76 files  ±0    76 suites  ±0   57s ⏱️ -1s
144 tests ±0  144 ✔️ ±0  0 💤 ±0  0 ±0 
452 runs  ±0  452 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit b346a73. ± Comparison against base commit 343b8bf.

@bmarty
Copy link
Member

bmarty commented Feb 17, 2022

Looks like we are reverting what have been done for #1384. The filter icon was not clear for the users.

Happy to accept the change, but I want to be sure that @daniellekirkwood has seen #1384

@bmarty bmarty disabled auto-merge February 17, 2022 10:53
@mnaturel
Copy link
Contributor Author

mnaturel commented Feb 17, 2022

Looks like we are reverting what have been done for #1384. The filter icon was not clear for the users.

Happy to accept the change, but I want to be sure that @daniellekirkwood has seen #1384

Okay no problem. I have kept my work of moving the filter bar in the lists if needed.

@github-actions
Copy link

Matrix SDK

Integration Tests Results:

  • [org.matrix.android.sdk.session]
    passed="25" failures="17" errors="0" skipped="2"
  • [org.matrix.android.sdk.account]
    passed="5" failures="0" errors="0" skipped="2"
  • [org.matrix.android.sdk.internal]
    passed="158" failures="1" errors="0" skipped="38"
  • [org.matrix.android.sdk.ordering]
    passed="16" failures="0" errors="0" skipped="0"
  • [org.matrix.android.sdk.PermalinkParserTest]
    passed="2" failures="0" errors="0" skipped="0"

@daniellekirkwood
Copy link
Contributor

Have seen the comments and the previous discussion. Let's stay accurate (filter icon) and implement a search feature as soon as we can

@bmarty bmarty merged commit ec2021d into develop Feb 17, 2022
@bmarty bmarty deleted the feature/mna/4643-filter-icon branch February 17, 2022 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Z-WTF WTF moment: High Impact, Low Effort
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants