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

Accessibility of the status page #3148

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Andrea-Guevara
Copy link

References

Description

Using grid system classes, creation of a new class in scss and media query to improve the responsiveness of the status page using the "item-operation" component.

Instructions for reviewers

List of changes in this PR:

  • In the "item-operation" component, the grid system classes "col-3" and "col-9" have been replaced by "col-12 col-md-3" and "col-12 col-md-9" in the div containing the text describing the functionality of the buttons and in the div containing the buttons.
  • The "item-operation.component.scss" file was created for styling.
  • The "column-height" class was created to define the height of the grid columns on larger and smaller screens using media queries.

To reproduce:

  • Log in as an administrator.
  • Enter an item and click on the edit button.
  • Click on the status button.
  • Zoom out and see that the texts and buttons on the status page are responsive.

…status page using the item-operation component
@tdonohue tdonohue added accessibility 1 APPROVAL pull request only requires a single approval to merge port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release bug labels Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 APPROVAL pull request only requires a single approval to merge accessibility bug port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release
Projects
Status: 🙋 Needs Reviewers Assigned
Development

Successfully merging this pull request may close these issues.

[Deque Analysis] Item Edit "Status" Tab "serious" accessibility issues
2 participants