Skip to content
This repository has been archived by the owner on Mar 9, 2024. It is now read-only.

🐞 Edit Post in Visual is "blank" and mce-editor button is stacked (one under other) #284

Open
byrevx opened this issue Nov 16, 2023 · 4 comments
Labels
status: needs triage This issue needs revision, splitting, or other gardening work type: bug Something isn't working

Comments

@byrevx
Copy link

byrevx commented Nov 16, 2023

Expected behavior

Edit Post in Visual must have text visible and editor button must be aligned to the left.

Current behavior

Edit Post in Visual is "blank" (hidden) and mce-editor button is left-stacked (one under other)
I try with Firefox and Chrome, same problem.

Possible solution

Edit Post in Visual is "blank" and after debuging I see inline-css:
.mce-panel {visibility: hidden;}
to solve I add in wp-includes/css/editor.min.css and wp-includes/css/editor.css
.mce-panel {visibility: visible;}

To solve mce button stacked (one under other) if using Visual Editor I add in wp-includes/css/editor.min.css and wp-includes/css/editor.css
.mce-btn {float: left;}

Steps to reproduce bug

Edit post and try to switch from Text to Visual!

Context

Changing wp-includes/css/editor.css it is the simplest and fastest solution that I have found. I do not know in detail about editor!

ClassicPress version

2.0.0+nightly.20231115

PHP version

7.4

Can you help?

I can help test a solution

@byrevx byrevx added status: needs triage This issue needs revision, splitting, or other gardening work type: bug Something isn't working labels Nov 16, 2023
@KTS915
Copy link
Member

KTS915 commented Nov 16, 2023

I am not seeing this, but I'm using PHP 8.2.

@byrevx
Copy link
Author

byrevx commented Nov 17, 2023

Tested now with 8.0 and 8.1 , same problem.

Text mode;
2023-11-17 10_51_48-Add New Post ‹ Stirile Altfel - Scurt Pe Doi — ClassicPress — Mozilla Firefox

Visual Mode:
2023-11-17 10_52_19-Add New Post ‹ Stirile Altfel - Scurt Pe Doi — ClassicPress — Mozilla Firefox

Dubugging:
2023-11-17 10_54_26-Edit Post “Test” ‹ Stirile Altfel - Scurt Pe Doi — ClassicPress — Mozilla Firefo

Removing inline css visibility: hidden; from mceu_23 (id) the information and buttons are visible, except that they are arranged vertically, they are not aligned to the left, but but adding in includes/css/editor.css the following rule .mce-btn {float: left;} everything is ok.

@viktorix
Copy link
Member

@byrevx thanks for reporting. Are there any errors in the console?

@byrevx
Copy link
Author

byrevx commented Nov 17, 2023

@byrevx thanks for reporting. Are there any errors in the console?

Uh, thanks for giving me a tip, it didn't occur to me to look in the console. It seems that I had a mixed context, http with https, it was a newly installed site and I use Cloudflare for ssl.

ClassicPress Address (URL) and Site Address (URL) were both with http, I changed to https and added to wp-config.php

$_SERVER['SERVER_PORT'] = 443;
$_SERVER['REQUEST_SCHEME'] = "https";
$_SERVER['HTTPS']='on';

of the 3 lines above, one must remain:
$_SERVER['SERVER_PORT'] = 443;
or
$_SERVER['HTTPS']='on';

otherwise I can't log in.

  • Apparently, on a site with the old version 1.4.4, authentication does not require changes in wp-config.php.

So theoretically it's not really a bug, it depends on the fact that the browser no longer allows mixed content, theoretically it's my mistake.

P.S. It would be useful to have a notice-warning of the "nag" type that would display after installation, in the dashboard, the fact that mixed content (http + https) is being used, as well as displaying a link (help) or information on how to solve the problem.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: needs triage This issue needs revision, splitting, or other gardening work type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants