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.

Sign up for GitHub

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

Jump to bottom

Charts overlapping with UI elements for gallery #337

Closed
dan-bowerman opened this issue May 11, 2023 · 1 comment · Fixed by #354
Closed

Charts overlapping with UI elements for gallery #337

dan-bowerman opened this issue May 11, 2023 · 1 comment · Fixed by #354
Assignees
@RyanCoulsonCA
Labels
Bug Something that probably needs fixing Priority: High This task is high priority and should be tackled soon Quality Assurance Tasks related to QA assessments Storylines Viewer Work surrounding the Storylines Viewer that loads individual StoryRAMP products

Comments

@dan-bowerman
Copy link
Member

Found by: Olivieri,Celina (elle, la | she, her)

  1. Product: Academic Challenge: NPRI Academic Challenge - Canada.ca (ec.gc.ca)
  2. Language: Both
  3. Issue Description: Table data expands over graph switch
  4. Steps to Reproduce:
    a. Scroll to 'University of Québec in Abitibi-Témiscamingue' chapter
    b. Click hamburger menu then Show Table Data
    c. Expected: I expect the displayed table data to be fully visible and not be covered
    d. Actual:
    e. ^ The last row of the table is covered by choosing different tables
    Also: columns don't seem to have any padding.

image

Feng,Yi Lei
Looks like on mobile the datatable will be completely cut off, so we will need to adjust the chart size or just turn off this option on mobile resolutions.
@dan-bowerman dan-bowerman added Storylines Viewer Work surrounding the Storylines Viewer that loads individual StoryRAMP products Quality Assurance Tasks related to QA assessments Bug Something that probably needs fixing Priority: High This task is high priority and should be tackled soon labels May 11, 2023
@dan-bowerman
Copy link
Member Author

Related #336

@dan-bowerman dan-bowerman mentioned this issue May 16, 2023
Open
@RyanCoulsonCA RyanCoulsonCA self-assigned this May 18, 2023
@RyanCoulsonCA RyanCoulsonCA mentioned this issue May 24, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RyanCoulsonCA RyanCoulsonCA

Labels
Bug Something that probably needs fixing Priority: High This task is high priority and should be tackled soon Quality Assurance Tasks related to QA assessments Storylines Viewer Work surrounding the Storylines Viewer that loads individual StoryRAMP products
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

remove view data table option from highcharts RyanCoulsonCA/story-ramp
2 participants
@dan-bowerman @RyanCoulsonCA