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(CodeHighlighting): ensure a scrollbar is shown when not enough space is available (pre tag) #1230

Merged
merged 1 commit into from
Jan 27, 2022

Conversation

tujoworker
Copy link
Member

I completely missed the correct tag. The scrollbars / overflow should have been added to the pre tag and not the code tag πŸ€¦β€β™‚οΈ

…ace is available (pre tag)

I completely missed the correct tag. The scrollbars / overflow should have been added to the pre tag and not the code tag πŸ€¦β€β™‚οΈ
@tujoworker tujoworker requested a review from dinarosv January 27, 2022 09:02
@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit d624c20:

Sandbox Source
eufemia-starter Configuration

@gatsby-cloud
Copy link

gatsby-cloud bot commented Jan 27, 2022

Gatsby Cloud Build Report

DNB Eufemia Portal

πŸŽ‰ Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

πŸ• Build time: 6m

Performance

Lighthouse report

Metric Score
Performance πŸ”Ά 66
Accessibility πŸ’š 100
Best Practices πŸ’š 100
SEO πŸ’š 92

πŸ”— View full report

Copy link
Contributor

@dinarosv dinarosv left a comment

Choose a reason for hiding this comment

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

Oh! I also didn't notice this in the last PR πŸ˜„

@tujoworker tujoworker merged commit 4bd2930 into main Jan 27, 2022
@tujoworker tujoworker deleted the fix/code-element-scroll branch January 27, 2022 12:01
tujoworker pushed a commit that referenced this pull request Feb 2, 2022
# [9.20.0](v9.19.0...v9.20.0) (2022-02-02)

### Bug Fixes

* add stylis plugin TS Type definitions ([4c8210c](4c8210c))
* **AvatarGroup:** remove left border from first avatar ([#1254](#1254)) ([bc1a870](bc1a870))
* **Breadcrumb:** fix React Node usage in aria-label ([1c921fa](1c921fa))
* **BreadCrumb:** make isCollapsed prop typing strict to only use boolean ([392e6eb](392e6eb))
* **CodeHighlighting:** ensure a scrollbar is shown when not enough space is available (pre tag) ([#1230](#1230)) ([4bd2930](4bd2930))
* **InfoCard:** change text props to React.ReactNode ([#1253](#1253)) ([79c37af](79c37af))
* label of avatar group to be react node ([#1244](#1244)) ([04df401](04df401))
* label of tag group to be react node ([#1245](#1245)) ([4949d85](4949d85))
* make Eufemia "warn" log better recognizable ([#1231](#1231)) ([c917b94](c917b94))
* **StepIndicator:** fix height animation issue when CSS is loaded too slow ([#1221](#1221)) ([8dfdd7e](8dfdd7e))
* **TagGroup:** make TagGroup component inline, so it can be used in body text / paragraphs ([#1229](#1229)) ([b64f5d3](b64f5d3))
* **TimeLine:** display skeleton when items is passed as children ([#1255](#1255)) ([4eaea6b](4eaea6b))
* **Timeline:** fix TypeScript issue when using children ([0a9571c](0a9571c))
* **Types:** correct usage of icon types inside Button and BreadcrumbItem ([85dee9c](85dee9c))
* **Types:** correct usage of title and size attributes along side with some other colliding props ([540998a](540998a))
* **Types:** fix SkeletonShow type usage ([03c8525](03c8525))

### Features

* add usePropsWithContext hook for hook based components ([#1234](#1234)) ([d60feeb](d60feeb))
* remove type="module" in order omit the "fullySpecified" spec ([bc3f12a](bc3f12a))
* **TypeScript:** enhance overall typing and add verify check ([608bede](608bede))
@tujoworker
Copy link
Member Author

πŸŽ‰ This PR is included in version 9.20.0 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants