Deep links return fragments in argument value [180042703]

Fixed

Bug

Status Update

No update yet.

Description

[Deleted User]

created issue #1

Feb 12, 2021 10:28PM

Component used: Navigation

Version used: 2.3.3

Devices/Android versions reproduced on: Pixel 3a, Samsung Galaxy S10

If a deep link contains a fragment, we get the fragment value in addition to the expected path. You can reproduce this with the basic navigation sample by using this deeplink: www.example.com/user/morgan#foo. See attached screenshot.

From Ian:

I suspect it could be as easy as catching both ? and # in https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:navigation/navigation-common/src/main/java/androidx/navigation/NavDeepLink.java;l=153

Instead of just ?

nav-sample-fragment.png

79 KB

View

Download

Comments

il...@google.com <il...@google.com> #2Feb 12, 2021 10:32PM

Assigned to jb...@google.com.

Nested scrolling works partially (as per

http://b/122818889). Let's discuss if we need full support and if so make sure it works.

jb...@google.com <jb...@google.com> Oct 6, 2021 04:52PM

Reassigned to sa...@google.com.

jb...@google.com <jb...@google.com> Nov 16, 2021 01:09AM

Reassigned to jb...@google.com.

ap...@google.com <ap...@google.com> #3Nov 16, 2021 05:27PM

Hi!

What is 'partially' exactly?
How do I see it?
Thanks!

jb...@google.com <jb...@google.com> #4Nov 16, 2021 05:56PM

Marked as fixed.

As of now:
- Nesting scroll views with a scroll direction perpendicular to the ViewPager2's orientation inside ViewPager2 works
- Nesting scroll views with a scroll direction parallel to the ViewPager2's orientation inside ViewPager2 does not work

Issue 180042703