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

[React Branch] QA Feedback Master Issue #1462

Open
clpetersonucf opened this issue Mar 17, 2023 · 27 comments
Open

[React Branch] QA Feedback Master Issue #1462

clpetersonucf opened this issue Mar 17, 2023 · 27 comments
Labels
React Branch Related to the React rewrite for Materia

Comments

@clpetersonucf
Copy link
Member

Figured it'd be easier to create a single issue for people to submit feedback based on testing Materia v10 on QA. I can stub out issues based on items people submit.

@clpetersonucf clpetersonucf added the React Branch Related to the React rewrite for Materia label Mar 17, 2023
@clpetersonucf
Copy link
Member Author

clpetersonucf commented Mar 17, 2023

  1. Widget request pagination is fairly slow in My Widgets, and makes it pretty clear 10 per request is far too few. That number could safely be bumped up considerably.
  2. The -cores aren't loading properly, which is expected. NGINX dev was providing a redirect, but the easier thing is to probably scrap putting compiled assets in public/dist and just emit them straight into the public directory, to better match where those assets were located before.
  3. We still have to fix the issue where collaborators are being displayed in duplicate due to a variable type change from the PHP 8.1 update.
  4. Views that should be sourced from Materia-Theme-UCF are not rendering yet (note: this is resolved, but the resolution is in the private UCF docker-compose that's not within ucfopen.)

@FrenjaminBanklin
Copy link
Contributor

  1. The splash screen has an ever-present horizontal scrollbar that seems to be the result of a few elements using the width: 100vw; rule combined with a hidden image that's over 1000 pixels wide.
  2. The widget creator page doesn't seem to be handling vertical size well, but the impact seems to vary from widget to widget. Generally, if the creator is not tall enough then whatever horizontal scroll bar is available inside the iframe is not adequate to view everything.

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Mar 28, 2023

  1. The creator Save History dialog's cancel button is operable
  2. The widget icon alignment in the detail page is skewed

Screenshot 2023-03-28 at 12 10 42 PM

  1. The "Return to My Widgets" link in the creator header, when editing a widget, is constructing a malformed URL

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Apr 5, 2023

  1. Embed code isn't generating correct width/height values for a given widget instance
  2. Embedded widgets may not be initializing properly - this should be tested with live instances, and not just local dev instances embedded in a local html file.

@clpetersonucf
Copy link
Member Author

New issues discovered in the apha4 build:

  • If a widget hash is present in the URL, the widget will not be selected when the instance list is fully loaded
  • Copying or deleting a widget reloads the widget list, but the filter to sort them by creation date does not appear to be operating on the list
  • The 'or create a new widget' button appears to be below the viewport of the LTI picker window in Canvas:

Screenshot 2023-04-20 at 1 51 14 PM

@cayb0rg
Copy link
Contributor

cayb0rg commented May 3, 2023

  • Searching by creation date/time doesn’t work in the admin/support pages
  • (Missing feature) Cannot see the owner of a widget in admin/support pages (could have been omitted for privacy?)
  • Help/Support contact details missing
  • Widget navigation in My Widgets disappears on smaller screen widths (<980px):

Screenshot 2023-05-03 at 7 53 16 AM

@clpetersonucf
Copy link
Member Author

clpetersonucf commented May 4, 2023

  • Guest scores are not populating under Student Activity if a widget has Guest Mode enabled
  • The score screen's "play again" option doesn't display in some circumstances where it should (ex: an instance has attempt limits enabled, but the student has remaining attempts available to them)
  • draft_not_playable page not rendering properly

@clpetersonucf
Copy link
Member Author

clpetersonucf commented May 19, 2023

Resolved @cayb0rg's feedback items:

Searching by creation date/time doesn’t work in the admin/support pages

There wasn't a good way to implement this, so the language suggesting this as an option has been updated.

(Missing feature) Cannot see the owner of a widget in admin/support pages (could have been omitted for privacy?)

Instance owners are now displayed in instance admin. Also added instance IDs and user IDs as URL hashes for each admin page, allowing direct navigation to a user or instance management if you know their ID.

Help/Support contact details missing

This requires enabling an additional environment variable, which is disabled by default. However, some locations where support information should be displayed may not display the overridden support details yet.

Widget navigation in My Widgets disappears on smaller screen widths (<980px):

The media query that causes this has been removed, pending a better implementation of responsiveness on the My Widgets page.

@clpetersonucf
Copy link
Member Author

clpetersonucf commented May 26, 2023

Style improvements:

  • The formatting of the 500 error page has some issues
  • The "Are you sure you want to edit this widget?" should be reevaluated for clarity and necessity, with some small styling adjustments required regardless
  • Screenshots on the home page are lossy and low resolution and should be updated

@cayb0rg
Copy link
Contributor

cayb0rg commented May 31, 2023

Collaborator Issues

  • Users with view access cannot remove themselves as a collaborator. Not sure if this is a bug or if the option to delete themselves should be removed entirely.
  • Users cannot re-add themselves or super users as collaborators of a widget through the admin instance panel.

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Jun 6, 2023

The score screen is making a call to apiGetWidgetInstanceScores when in the context of reviewing another user's score from My Widgets, which is a behavior not present on production. This causes the score screen to error out in some situations due to permission mismatches, such as when the user has View Scores access to the widget.

@clpetersonucf clpetersonucf pinned this issue Jun 6, 2023
@clpetersonucf
Copy link
Member Author

  • Notifications that inform the user about shared widgets should correctly link to the widgets.
  • Certain My Widgets' page behaviors are still breaking when copying or deleting instances. Copying a widget forces the instance list to reload, and the newly created instance is not selected. Similarly, deleting a widget forces the instance list to reload, but the widget list is not displayed in chronologically descending order.

@cayb0rg
Copy link
Contributor

cayb0rg commented Jun 13, 2023

Not an issue, but having the option to see the created_at date and possibly sort by or search for date in My Widgets could be useful, since we're already sorting by that value.

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Aug 10, 2023

The widget catalog does not appear to filter widgets by the in_catalog flag.

@dmols
Copy link

dmols commented Aug 11, 2023

What I've noticed thus far:

  • Alignment is really off for the Galaxy Fold, and get-started content div doesn't extend all the way across the screen. Might be best to leave it as is though, since even the landing page for the Fold doesn't have correct display.
  • Some widgets have grammatical errors: "Hiring Decision Making Mulitple" is one of them
  • 'Mobile Friendly' widgets that are not mobile accessible (wrong screen sizing): DNA Sequencer, Dodgeball, Enigma, Flashcards, Last Chance Cadet, etc
  • MyWidgets also doesn't display correctly

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Aug 11, 2023

@dmols some more specificity is useful here:

Might be best to leave it as is though, since even the landing page for the Fold doesn't have correct display.

What model of Fold? The cover screen or interior screen? Knowing those it's at least possible to emulate the display size using dev tools to identify styling issues.

MyWidgets also doesn't display correctly

How so?

@cayb0rg
Copy link
Contributor

cayb0rg commented Aug 11, 2023

  • When embedding a widget's iframe into a page, there are scrollbars (maybe due to the Materia footer). The user can just make the width and height attributes slightly larger, but this may not be obvious
  • refetchInstances in select-item.jsx is not a function
  • Legacy urls (https://localhost/lti/assignment?widget=[id]) do not load in LTI Assignment Launch, dunno if these are necessary or still work in some cases
  • Finishing a widget (new or old) that counts for points causes the alert "Something went wrong... : Invalid play session : is fatal" to appear. It still saves the score in Materia, just not Webcourses. Didn't get the error with widgets that have 0 points.

@dmols
Copy link

dmols commented Aug 11, 2023

@clpetersonucf , here are some extra details on my review!

What model of Fold? The cover screen or interior screen? Knowing those it's at least possible to emulate the display size using dev tools to identify styling issues.

This I'm not entirely sure about since I haven't been able to test every model, but I assume the screen size is the same for every model, since users online have mentioned having issues with any and all. Also the dev tools for Google Chrome only indicate one option for the 'Fold'.

MyWidgets also doesn't display correctly

Apologies for not elaborating fully; the page displays but it's extremely zoomed in, despite what device it's displayed on, even the standard 'responsive' option. The full page does not fully show when using the dev tools, and minimizing the window down to a certain range (wish I knew how to grab the exact pixel width but inspecting seems to make the issue worse) cuts off the right edge of the content.

@clpetersonucf
Copy link
Member Author

The footer can become visually decoupled from the widget player when the embedded iframe scales responsively (this was embedded as an assignment in Webcourses):
Screenshot 2023-08-15 at 2 31 47 PM

@cayb0rg
Copy link
Contributor

cayb0rg commented Aug 31, 2023

  • Hash changes while on the My Widgets Page do not redirect to that widget (such as when you click on a link in a notification)
  • No error given when attempting to add a student with only the support_user role (no basic_author role) when guest mode is disabled. It doesn't save (as expected), but it also doesn't raise any errors.
    • The error occurs here when checking whether a user is a student. We can check whether a student is also a support user or return an error telling the user to enable guest mode.
  • Last name overlaps role in Collaborator:
image
  • Making a copy of a widget in Instance Admin sets the hash to an object

@clpetersonucf
Copy link
Member Author

@cayb0rg just a heads up I have fixes for 1 and 4 in #1492

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Sep 5, 2023

  • The catalog is duplicating featured widgets, including them in the featured section and then again in the non-featured section.
    Resolved. For posterity, in_catalog determines whether a widget is featured, is_playable determines whether a widget will be displayed in the catalog at all.

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Sep 5, 2023

  • Widget previews do not provide a Preview Again button.
  • Score screens periodically fail to render due to an error originating from drawScoreCircle(). This is likely due to a race condition.
  • The above error occurs when rendering custom score screens as well, which makes one wonder why that function is being called at all when a custom score screen is provided.
  • Custom score screens are very slow to initially render.

@cayb0rg
Copy link
Contributor

cayb0rg commented Sep 14, 2023

More for quality of life, but it could be helpful to prefill the course ID in the extra attempts dialog with the current selected instance's ID since we're already passing in inst.

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Sep 22, 2023

  • Enigma widgets don't proceed past the "Submit for review" screen on QA when embedded as assignments in Webcourses. They do appear to be marked as complete, however (EDIT: this appears to be due to an edge case in the widget-player when play_logs_save returns a score url in addition to the score, in addition to an overlap in timing for when logs are submitted. I'm working on untangling this code a bit and implementing a fix.)
  • The "Previewing Prior Save" action bar overlay in the creator, activated when viewing an older qset from the Save History dialog, has style issues in Safari. The buttons overflow, rendering the Cancel button unclickable.

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Sep 22, 2023

  • Score screen overviews are still not updating properly when selecting a different previous attempt (I swear to god I fixed this)

@clpetersonucf
Copy link
Member Author

clpetersonucf commented Sep 29, 2023

  • Viewing single scores is not conditionally disabling the header when IS_EMBEDDED is provided as a window value
  • The score screen still enters a failure state when viewing single scores for widgets (for example, in the Canvas speed grader) that the current user does not have access to
  • The confirmation dialog triggered from beforeunload should not be displayed in situations where the player has errored out (such as an invalid heartbeat)

@clpetersonucf clpetersonucf unpinned this issue Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
React Branch Related to the React rewrite for Materia
Projects
None yet
Development

No branches or pull requests

4 participants