Skip to content

Adr2/update report html UI#506

Merged
astenstrasser merged 7 commits intoscanapi:mainfrom
GitVitor:ADR2/update-report-HTML-ui
Apr 10, 2022
Merged

Adr2/update report html UI#506
astenstrasser merged 7 commits intoscanapi:mainfrom
GitVitor:ADR2/update-report-HTML-ui

Conversation

@GitVitor
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

Pradhvan commented Oct 6, 2021

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

@GitVitor
Copy link
Copy Markdown
Contributor Author

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

Copy link
Copy Markdown
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?

Comment thread scanapi/templates/report.html Outdated
Comment thread scanapi/templates/report.html Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md
@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.

4 participants