Skip to content

fixed the greying image issue that occurs after uploading jpg image #2603 #2894

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

Conversation

aditya123473892
Copy link

Fixes #issue-number

Changes: This will fix the error of resizing the window uploaded image gets greyed out and shifts to bottom #2603

I have verified that this pull request:

has no linting errors (npm run lint)
has no test errors (npm run test)
is from a uniquely-named feature branch and is up to date with the develop branch.
is descriptively named and links to an issue number, i.e. Fixes #2603

@lindapaiste
Copy link
Collaborator

I wonder if there are any unintended consequences from applying the entire desktop .editor-holder style to the mobile version?

The part that we need is:

&.editor-holder--hidden .CodeMirror {
display: none;
}

The part that I'm concerned about is:

.editor-holder {
height: calc(100% - #{29 / $base-font-size}rem);
width: 100%;
position: absolute;
@include themify() {
border: 1px solid getThemifyVariable("ide-border-color");
}

@lindapaiste lindapaiste added the Enhancement Improvement to an existing feature label Jan 14, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@raclim
Copy link
Collaborator

raclim commented Jun 6, 2024

Thanks for your work on this! I don't think this solution succinctly addresses the issue here, so I'm going to close this for now. I'm sorry that we couldn't get this in, but please feel free to create another PR with an updated solution or check out our other issues!

@raclim raclim closed this Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When resizing the window uploaded image gets greyed out and shifts to bottom
3 participants