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

[Test failure] TC_LEARNER_31: Faint unexpected rectangle #345

Closed
Colin-Fredericks opened this issue May 17, 2024 · 4 comments
Closed

[Test failure] TC_LEARNER_31: Faint unexpected rectangle #345

Colin-Fredericks opened this issue May 17, 2024 · 4 comments
Labels
redwood release testing Affects the upcoming release (attention needed)
Milestone

Comments

@Colin-Fredericks
Copy link

Release

Redwood

Expected behavior

Not 100% sure - this might actually be a design choice rather than a bug.

Actual behavior

There's a light-colored rounded rectangle behind and offset from the course image. I'm not 100% sure that's intentional - not sure whether it's supposed to be there or directly behind the image.

Screenshot 2024-05-17 at 9 47 22 AM

Steps to reproduce

Just visit any About page.

Additional information

In ones without an image you get a blank square in front of the rectangle.

Screenshot 2024-05-17 at 9 48 06 AM

Also - none of the test courses on the edly server have a video on their about page, so I couldn't test to see whether those work.

@Colin-Fredericks Colin-Fredericks changed the title [Test failure] <Test Case ID>: <DESCRIPTIVE TITLE> [Test failure] TC_LEARNER_31: Faint unexpected rectangle May 17, 2024
@jmakowski1123 jmakowski1123 added the release testing Affects the upcoming release (attention needed) label May 17, 2024
@mariajgrimaldi mariajgrimaldi added this to the Redwood.1 milestone May 17, 2024
@mariajgrimaldi
Copy link
Member

mariajgrimaldi commented May 20, 2024

Hello @Colin-Fredericks, thanks for the report! The sandbox is resetting, so I'll test this again later today and report back. Thank you! In the meantime, I'll provision a dev environment to test it locally.

@arbrandes: do you know whether this is the expected behavior for the course about page? I don't think so, but I want to confirm.

@mariajgrimaldi
Copy link
Member

mariajgrimaldi commented May 21, 2024

We have a plain (no themes) redwood installation, and there, and there are no faint rectangles in the course about page. So this might be related to the indigo theme used in the sandbox. I'll open a new issue there.

@mariajgrimaldi
Copy link
Member

According to this comment, the first screenshot is a design choice. However, the second is a bug, so they're now aware of it. Thanks!

@mariajgrimaldi mariajgrimaldi moved this from In progress to In review in Build-Test-Release Working Group May 30, 2024
@mariajgrimaldi
Copy link
Member

I tested this on https://sandbox.openedx.edly.io and it's working correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
redwood release testing Affects the upcoming release (attention needed)
Projects
Development

No branches or pull requests

3 participants