-
Notifications
You must be signed in to change notification settings - Fork 176
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
Hotfixes - Import from other channels - The 'Show more' button is not centered #4838
Comments
Hi @pcenov |
Hi @rishisulakhe, Thank you for your interest. I've assigned you this issue. Please follow these guidelines to get started, if you haven't already https://github.com/learningequality/studio?tab=readme-ov-file#how-can-i-contribute Let us know if you run into issues |
is this issue still open |
Hi @hsd2514, it's already assigned. You're welcome to find another 'help wanted' with no assignee. |
Hi @jayesh9747, thanks for volunteering. Many contributors are students or have other responsibilities, and we really don't expect to receive solution within days to not make contributing experience into pressure. Let's wait a couple of days to hear from @rishisulakhe whether this is in progress or no. If we don't hear back, I'm happy to reassign. |
@MisRob , okay sure. i give my quick introduction.I am Jayesh Savaliya, a third-year B.Tech student in Computer Science and Engineering at IIIT Pune, graduating in 2026. I have a strong interest in full-stack development, particularly in the MERN stack and Django backend framework, and have hands-on experience in Python for machine learning projects. My open-source contributions include Dalgo-admin and the OpenKruise-dashboard, and I have also interned under the C4GT mentorship program with ShikshaLokam.I am very keen to contribute. |
Welcome @jayesh9747! |
Hello @MisRob Is the issue fixed .I love to do this as my first issue. |
@rishisulakhe , are you still working ? otherwise i would to contribute this issue. |
@jayesh9747 , This issue can be fixed within a day .Isn't it ? |
Friends, as mentioned above, let's give some time to @rishisulakhe ;) I recommend to read existing issue comments before posting - you can see we decided to wait. We typically unassign after a month of inactivity. You're welcome to post an assignment request on free 'help wanted' issues and enjoy contributing in whatever pace works for you. |
@MisRob Thanks, for clarifying it . we will wait and focus on some other issues. |
Thanks @Md-Murthaza. There will be more new contributing opportunities by the end of January or so, and we regularly cleanup stale assignments and label new issues, so it's best to keep an eye on it regularly. |
Hi @jayesh9747, I will now re-assign to you because @rishisulakhe didn't respond and it's been a month since the assignment. Feel free to take work on this if you're still interested, thanks! Also I wanted to mention that Learning Equality will be closed from December 23 to January 5. |
@MisRob , i will complete very soon. |
@pcenov @MisRob , I believe the actual issue is that the 'Show More' button is not centered. Additionally, if I revisit the same page, the 'Show More' button does not appear. Moreover, in Level 2, there are 35 resources, but when revisiting the page, the 'Show More' button does not appear. I suspect this happens because the content is preloaded, resulting in only 25 documents being loaded, as shown in the first visit. This is also evident in the screenshot below: Finally, I’d like to ask: should I fix this issue within this thread, or should I create a new issue for it? |
@MisRob can you clarify this issue ? should I process or not ? |
Hey @jayesh9747! Thanks! It seems that you spotted an issue in our pagination process :). I dont see an open issue reporting this. So please feel free to open a new issue reporting this. |
@AlexVelezLl @MisRob please check Review PR#4864. Thank you. |
Observed behavior
The 'Show more' button is not centered at the 'Import from other channels' modal.
Expected behavior
The button should be centered for constancy as it's centered everywhere else.
Steps to reproduce the issue
Usage Details
The text was updated successfully, but these errors were encountered: