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

[4.1] Codemirror Full Screen #37129

Merged
merged 5 commits into from
Mar 3, 2022

Conversation

brianteeman
Copy link
Contributor

This PR re-introduces the on screen instruction how to make codemirror full screen. (No idea why it was removed the pr doesnt help) AND fixes the positioning of the editor when full screen so that the first 5 lines are not hidden.

As there is a css change you will need to use a pre-build package or npm run build:css

Easiest to test in com_templates as that uses codemirror by default

This PR re-introduces the on screen instruction how to make codemirror full screen. (No idea why it was removed the pr doesnt help) AND fixes the positioning of the editor when full screen so that the first 5 lines are not hidden.

As there is a css change you will need to use a pre-build package or `npm run build:css`
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.1-dev labels Feb 24, 2022
@Quy
Copy link
Contributor

Quy commented Feb 24, 2022

How about float right to be subtle?

37129-codemirror

@brianteeman
Copy link
Contributor Author

updated as requested

@Quy
Copy link
Contributor

Quy commented Feb 25, 2022

I have tested this item ✅ successfully on 95fd2bb


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37129.

1 similar comment
@jwaisner
Copy link
Member

I have tested this item ✅ successfully on 95fd2bb


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37129.

@jwaisner
Copy link
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37129.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 26, 2022
@bembelimen bembelimen merged commit cc800b0 into joomla:4.1-dev Mar 3, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 3, 2022
@bembelimen
Copy link
Contributor

Thx

@bembelimen bembelimen added this to the Joomla 4.1.1 milestone Mar 3, 2022
@brianteeman
Copy link
Contributor Author

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants