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

fix: updated css styling to prevent content spilling #655

Merged
merged 2 commits into from
Jan 30, 2024

Conversation

ayesha-waris
Copy link
Contributor

INF-891

Explanation

Styling for the code part of HTML has been added to prevent spilling out of the container.

Before Change:
299603738-40927ae9-0bf8-4d73-9ebf-e1cad9087e5a-2

After Change:

299603907-9dd7b9c6-cfd8-49ba-b1f8-23f501a07cc1-2

@ayesha-waris ayesha-waris changed the title INF-891 Explanation Styling for the code part of HTML has been added to prevent spilling out of the container. Before Change: Screenshot 2024-01-25 at 2 12 34 PM After Change: Screenshot 2024-01-25 at 2 13 36 PM fix: updated css styling to prevent content spilling Jan 25, 2024
Copy link

codecov bot commented Jan 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3aef3b0) 92.39% compared to head (649275f) 92.27%.

❗ Current head 649275f differs from pull request most recent head 1542d91. Consider uploading reports for the commit 1542d91 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #655      +/-   ##
==========================================
- Coverage   92.39%   92.27%   -0.12%     
==========================================
  Files         169      169              
  Lines        3456     3481      +25     
  Branches      896      897       +1     
==========================================
+ Hits         3193     3212      +19     
- Misses        243      249       +6     
  Partials       20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@sundasnoreen12 sundasnoreen12 left a comment

Choose a reason for hiding this comment

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

PR LGTM

@sundasnoreen12 sundasnoreen12 merged commit 88a985d into master Jan 30, 2024
5 checks passed
@sundasnoreen12 sundasnoreen12 deleted the eemaanamir/INF-891 branch January 30, 2024 07:59
ayesha-waris added a commit that referenced this pull request Jan 31, 2024
* fix: updated css styling to prevent content spilling

* style: small property change

---------

Co-authored-by: eemaanamir <eemaan.amir@gmail.com>
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