fix(musicRecommend): scroll-to-top button overlapped with content

[s25g5d4]

Description

Use a container to wrap the content, so that some paddings are added to avoid the content overlapping with the scroll-to-top button.

Related Issue

366

Motivation and Context

366

How Has This Been Tested?

tested with local dev server.

• [x] Chrome (Desktop)
• [x] Chrome (Mobile)
• [x] Firefox (Desktop)
• [x] Firefox (Mobile)
• [ ] Safari (Desktop, optional)
• [ ] Safari (iPhone, optional)
• [ ] Safari (iPad, optional)

Screenshots (if appropriate):

Additional Notes

This change makes the result table narrower. Alternatively, we can just add some padding at the bottom.

[dnaroma]

I will have access to my computer tomorrow and check this PR. Sorry for the delay.

[dnaroma]

I think making table narrow is not a good idea. I suggested two options:

1. limit the height to avoid overlapping
2. make scroll-to-top button invisible (maybe accomplished by 1 as well)

[s25g5d4]

sorry, I don't get what you mean "limit the height to avoid overlapping". limit the height of which element? and how does it solve the problem?

[s25g5d4]

I think we can simply add some space below the table. that could avoid the overlapping problem without losing the scroll-to-top button.

[dnaroma]

sorry, I don't get what you mean "limit the height to avoid overlapping". limit the height of which element? and how does it solve the problem?

scroll-to-top button will not show if page is not scrolled, and it is not necessary to have a such button here.

[dnaroma]

Sorry I made a mistake, I tought it is music meta page.