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: bad overflow in modal for content with very long strings #161

Merged
merged 2 commits into from
Apr 23, 2022

Conversation

CarmitKl
Copy link

@CarmitKl CarmitKl commented Apr 18, 2022

Set also a max-width and an 'overflow-wrap' so modal-content that has very long strings (like in some error messages) will wrap correctly. Yet, keep the centered style for short content.

Checklist

  • Changes have been done against master branch, and PR does not conflict
  • New unit / functional tests have been added (whenever applicable)
  • Test are passing in local environment
  • Docs have been updated
  • PR title is follow the Conventional Commits convention: <type>[optional scope]: <description>, e.g: fix: prevent racing of requests

This change is Reviewable

… very long strings (like in some error messages) will wrap correctly. yet, keep the centerized style for short content
@CarmitKl CarmitKl requested a review from dan-ziv April 18, 2022 13:34
@CarmitKl CarmitKl added the ready for review Pull request is ready to be reviewed label Apr 18, 2022
@dan-ziv dan-ziv changed the title fix: bad overflow in modalText for content with very long strings fix: bad overflow in modal for content with very long strings Apr 23, 2022
@dan-ziv dan-ziv merged commit 36a1c6f into dev Apr 23, 2022
@dan-ziv dan-ziv deleted the fix/modalText branch April 23, 2022 21:14
dan-ziv pushed a commit that referenced this pull request Apr 23, 2022
 ### [1.2.1-dev.4](v1.2.1-dev.3...v1.2.1-dev.4) (2022-04-23)

### 🔧 Fixes

* bad overflow in modal for content with very long strings ([#161](#161)) ([36a1c6f](36a1c6f))
@dan-ziv
Copy link
Collaborator

dan-ziv commented Apr 23, 2022

🎉 This PR is included in version 1.2.1-dev.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

dan-ziv pushed a commit that referenced this pull request May 2, 2022
 ## [1.2.0](v1.1.0...v1.2.0) (2022-05-02)

### 💉 Improvements

* move to hooks useConstants, useEnvs, useTranslation ([#159](#159)) ([b77ba11](b77ba11))
* useTracking ([#160](#160)) ([a8cad80](a8cad80))
* useTranslation hooks ([#162](#162)) ([92e0927](92e0927))

### 🧩 Features

* integrated with get-starknet-wallet ([#163](#163)) ([e0f996a](e0f996a))
* terms of use ([#141](#141)) ([8d9fe79](8d9fe79))
* validate starknet wallet chain ID ([#168](#168)) ([2551a7f](2551a7f))

### 🔧 Fixes

* accept button stayed disabled when browser zoom is different from 100% ([#158](#158)) ([1e4afac](1e4afac))
* add ETH mainnet addresses ([#167](#167)) ([56fa74b](56fa74b))
* app crashed when trying to connect while ArgentX wasn't installed ([55e25f7](55e25f7))
* bad overflow in modal for content with very long strings ([#161](#161)) ([36a1c6f](36a1c6f))
* calc L2 tx hash in the background ([#169](#169)) ([80d23d2](80d23d2))
* canceled transfers shown in transfer log ([#166](#166)) ([69324e8](69324e8))
* **deps:** bump gsw version ([6c01ae5](6c01ae5))
* **Faq:** question 8 ([#157](#157)) ([51da4aa](51da4aa))
* handle display of exponential balances ([44c675b](44c675b))
* isInstalled() check for gsw ([#165](#165)) ([1881192](1881192))
* unsupported chain id does not showing  ([#151](#151)) ([afa5294](afa5294))
* use getStarknet() provider on starknet contracts ([1d6b023](1d6b023))
* use wallet provider for all provider calls ([#170](#170)) ([646feee](646feee))
* **utils:** get url params returns undefined ([cde6401](cde6401))
dan-ziv pushed a commit that referenced this pull request May 2, 2022
 ## [1.3.0](v1.2.0...v1.3.0) (2022-05-02)

### 💉 Improvements

* move to hooks useConstants, useEnvs, useTranslation ([#159](#159)) ([b77ba11](b77ba11))
* useTracking ([#160](#160)) ([a8cad80](a8cad80))
* useTranslation hooks ([#162](#162)) ([92e0927](92e0927))

### 🧩 Features

* integrated with get-starknet-wallet ([#163](#163)) ([e0f996a](e0f996a))
* validate starknet wallet chain ID ([#168](#168)) ([2551a7f](2551a7f))

### 🔧 Fixes

* add ETH mainnet addresses ([#167](#167)) ([56fa74b](56fa74b))
* bad overflow in modal for content with very long strings ([#161](#161)) ([36a1c6f](36a1c6f))
* calc L2 tx hash in the background ([#169](#169)) ([80d23d2](80d23d2))
* canceled transfers shown in transfer log ([#166](#166)) ([69324e8](69324e8))
* **deps:** bump gsw version ([6c01ae5](6c01ae5))
* handle display of exponential balances ([44c675b](44c675b))
* isInstalled() check for gsw ([#165](#165)) ([1881192](1881192))
* use getStarknet() provider on starknet contracts ([1d6b023](1d6b023))
* use wallet provider for all provider calls ([#170](#170)) ([646feee](646feee))
* **utils:** get url params returns undefined ([cde6401](cde6401))
@dan-ziv
Copy link
Collaborator

dan-ziv commented May 2, 2022

🎉 This PR is included in version 1.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@dan-ziv dan-ziv restored the fix/modalText branch September 6, 2022 11:17
@dan-ziv dan-ziv deleted the fix/modalText branch September 6, 2022 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review Pull request is ready to be reviewed released on @dev released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants