Skip to content

Conversation

@futa-ikeda
Copy link
Contributor

@futa-ikeda futa-ikeda commented Nov 22, 2021

Purpose

  • Address various bugs found in Revisions workflow

Summary of Changes

  • Darken color contrast for create buttons (3.41 -> 5.33 : 1, passes AA)
  • Small wording changes
  • Auto-close the dropdown when selecting a revision on the overview page
  • Add data-test selectors to delete button modal for automated testing

Screenshot(s)

Side Effects

  • All create buttons will have better color contrast!

QA Notes

@futa-ikeda futa-ikeda changed the base branch from develop to release/21.10.0 November 22, 2021 19:54
@futa-ikeda futa-ikeda changed the title [ENG-3432] Revisions bugfixes [ENG-3243] Revisions bugfixes Nov 22, 2021
@coveralls
Copy link

coveralls commented Nov 22, 2021

Pull Request Test Coverage Report for Build 1496329879

  • 0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 73.127%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/osf-components/addon/components/registries/update-dropdown/component.ts 0 2 0.0%
Totals Coverage Status
Change from base Build 1482079344: 0.005%
Covered Lines: 5169
Relevant Lines: 6802

💛 - Coveralls

{{#if this.revisions}}
{{#each this.revisions as |revision index|}}
<Registries::UpdateDropdown::ListItem
@onRevisionSelect={{action this.onRevisionSelect dd.close}}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to do something more simple like @onClick={{dd.close}}, but doing so caused the page to do a full load every time. I don't think this is the best way of closing the dropdown on click, but other attempts have either run into the problem where it does a full app load as opposed to a transition, or doesn't show which revision is the currently selected one (like when using <OsfLink>

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any reason why it does a full page load ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not exactly sure why. I tried messing around with some different approaches, but basically anything that closed the dropdown before the transition finished would trigger a full page load. I'm guessing there's something about how the transition is being called that gets messy when the dropdown is closed, but I couldn't quite figure out how to remedy that.

@futa-ikeda futa-ikeda changed the title [ENG-3243] Revisions bugfixes [ENG-3243][ENG-3417] Revisions bugfixes Nov 23, 2021
@futa-ikeda futa-ikeda merged commit a62d10f into CenterForOpenScience:release/21.10.0 Nov 23, 2021
@futa-ikeda futa-ikeda deleted the revisions-bugfixes branch November 23, 2021 20:17
@adlius adlius added this to the 21.10.0 milestone Dec 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants