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

Adr2/update report html UI #506

Merged
merged 7 commits into from Apr 10, 2022

Conversation

GitVitor
Copy link
Contributor

@GitVitor GitVitor commented Sep 9, 2021

Description

UI change:
1. add Gotham Font Family;
2. Variables CSS with colors;
2.1. Update red color to use: #f14668;
2.2. Update green color to use: #48c78e;
3. Add visual wrapper to following areas: request content, request response and request test;
4. New endpoint status UI;

Motivation behind this PR?

Following recent discussions on #161 we decide to implement new details to help users on visualize related request data.

What type of change is this?

Feature

Checklist

  • I have added a changelog entry / my PR does not need a new changelog entry. Instructions.
  • I have added/updated unit tests. Instructions.
  • New and existing unit tests pass locally with my changes. Instructions
  • I have self-documented code my changes by adding docstring(s) and comment(s). Instructions
  • Current PR does not significantly decrease the code coverage and docstring coverage.
  • My code follows the style guidelines of this project.
  • I have run ScanAPI locally and manually tested my changes. Instructions.
  • I have squashed my commits. Instructions.

Issue

#161

update report classes following BEMCSS pattern
close unclosed html tags
UI details include:
1. add Gotham Font Family;
2. Variables CSS with colors;
	2.1. Update red color to use: #f14668;
	2.2. Update green color to use: #48c78e;
3. Add visual wrapper to following areas: request content, request response and request test;
4. New endpoint status UI;
@GitVitor GitVitor requested review from a team as code owners September 9, 2021 02:05
@camilamaia
Copy link
Member

camilamaia commented Sep 9, 2021

Hey, @GitVitor, thank you very much for the PR 🙌

I run the branch on my local machine and tested it using Chrome, Firefox, and Safari.

I made a circle in the improvements points I found. In Safari, there is some information missing, for example. There are other small issues related to alignment, space...

Let me know what do you think!

Chrome

Version 93.0.4577.63 (Official Build) (x86_64)

Chrome
chrome-extended

Firefox

91.0 (64-bit)

firefox
firefox-expanded

Safari

Version 14.1.2 (16611.3.10.1.6)

safari
safari-expanded

@Pradhvan
Copy link
Member

Pradhvan commented Oct 6, 2021

@GitVitor hey 👋🏾 are you still working on this ?

@GitVitor
Copy link
Contributor Author

Hello folks, sorry i was busy in last month. I will work on this in next days.

Copy link
Member

@astenstrasser astenstrasser left a comment

Choose a reason for hiding this comment

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

Hey 👋
I noticed the Gotham Font change is the reason why layout is braking.
Maybe we could merge without changing the font for now?

scanapi/templates/report.html Outdated Show resolved Hide resolved
scanapi/templates/report.html Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@astenstrasser astenstrasser merged commit 36b5602 into scanapi:main Apr 10, 2022
astenstrasser added a commit that referenced this pull request Apr 10, 2022
* update report summary test location

update report classes following BEMCSS pattern
close unclosed html tags

* feat: add new UI details to report HTML

UI details include:
1. add Gotham Font Family;
2. Variables CSS with colors;
2.1. Update red color to use: #f14668;
2.2. Update green color to use: #48c78e;
3. Add visual wrapper;
4. New endpoint status UI;

* fix: revert font change

* Update CHANGELOG.md

* Update CHANGELOG.md

Co-authored-by: Alexandra Monteiro Stenstrasser <astenstrasser@gmail.com>
@astenstrasser astenstrasser mentioned this pull request Apr 12, 2022
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.

None yet

4 participants