Change 801715 had a related patch set uploaded (by Sbisson; author: Sbisson):

[mediawiki/extensions/Wikistories@master] Scroll select frame thumbnail into view

https://gerrit.wikimedia.org/r/801715

Change 801715 merged by jenkins-bot:

[mediawiki/extensions/Wikistories@master] Scroll select frame thumbnail into view

https://gerrit.wikimedia.org/r/801715

Testing:

@SBisson

1. The horizontal scroll bar shows when there are story pages that are more than the page can display, depending on the display size.

2. When the user scrolls, the horizontal scroll bar stops being visible. I think it should always be visible, to show there is more information on either side of the user's view.

This task is about scrolling the newly added frame's thumbnail into view.

Scrollbar visibility is controlled by the browser in most cases. The trend in recent years has been to not show them as much as possible.

When I select the "replace this image", after one image is selected, the page auto scroll down to the bottom

I think this is unexpected behaviour.

Before selecting "replace this image"	After selecting

For reference: https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollIntoView

Change 803270 had a related patch set uploaded (by Sbisson; author: Sbisson):

[mediawiki/extensions/Wikistories@master] Scroll the current frame into view only horizontally

https://gerrit.wikimedia.org/r/803270

Patch for the vertical scrolling bug is ready for code review

Change 803270 merged by jenkins-bot:

[mediawiki/extensions/Wikistories@master] Scroll the current frame into view only horizontally

https://gerrit.wikimedia.org/r/803270

Hi @SBisson ,

The vertical scrolling bug is gone. I noticed something about the horizontal scroll:

1. Add one image: the newly added thumbnail is partially brought into view. I think it should be brought fully into view with no scroll horizontal scroll space.

2. Add more than 1 image: only the first image of the set is brought into view. I wonder if it would be better to show the last image, so that all new images are seen? I also understand this means the user will have to work backwards to add text, which may not be so great. Just a thought.

Change 804603 had a related patch set uploaded (by Sbisson; author: Sbisson):

[mediawiki/extensions/Wikistories@master] Bring newly added frame completely into view

https://gerrit.wikimedia.org/r/804603

In T309128#7991522, @EUdoh-WMF wrote:

Hi @SBisson ,

The vertical scrolling bug is gone. I noticed something about the horizontal scroll:

1. Add one image: the newly added thumbnail is partially brought into view. I think it should be brought fully into view with no scroll horizontal scroll space.

I made a small follow-up patch to make this better. (moving back to code review)

2. Add more than 1 image: only the first image of the set is brought into view. I wonder if it would be better to show the last image, so that all new images are seen? I also understand this means the user will have to work backwards to add text, which may not be so great. Just a thought.

I'm not sure about that. I'll let @SGautam_WMF check when it's in design review.

Change 804603 merged by jenkins-bot:

[mediawiki/extensions/Wikistories@master] Bring newly added frame completely into view

https://gerrit.wikimedia.org/r/804603

Thanks for the fix. Looks great.

@SGautam_WMF can this task be resolved?

2. Add more than 1 image: only the first image of the set is brought into view. I wonder if it would be better to show the last image, so that all new images are seen? I also understand this means the user will have to work backwards to add text, which may not be so great. Just a thought.

I'm not sure about that. I'll let @SGautam_WMF check when it's in design review.

@EUdoh-WMF Thanks for sharing this point, current approach seems to be working ok after playing with different scenarios(add only 1 image to update 8 images). We can wait to see if there is more feedback on this particular point.