Skip to content
This repository has been archived by the owner on Jun 23, 2022. It is now read-only.

Neo2 theme doesn't work with pipeline editor #2

Closed
johnarnold opened this issue Nov 7, 2017 · 7 comments
Closed

Neo2 theme doesn't work with pipeline editor #2

johnarnold opened this issue Nov 7, 2017 · 7 comments

Comments

@johnarnold
Copy link

Hi Tobi,

Thanks for your work on neo2 theme. Mostly the theme works great, but there is one issue that persists.

When creating pipeline jobs, the job editor doesn't display text correctly. The curser does not display in the correct position in the text box, it's always some spaces to the right of where you're actually editing. this makes the editor basically unusable.

This issue was reported here:
https://issues.jenkins-ci.org/browse/JENKINS-36475
jenkins-contrib-themes#19

It seems that the ACE text editor plugin doesn't support non-monospaced fonts. Is it possible to tweak your css so that it uses a monospace font when the class is ace_content, as suggested by @slide ?

Or can you think of another fix?

Thanks!

@johnarnold
Copy link
Author

Hmm seems like you've already seen this issue. Any idea why this commit doesn't fix it?
ab1d9d0

@slide
Copy link

slide commented Nov 7, 2017

Is it possible that the files in dist don't have that fix? I cloned the repo and built the case myself, I'll test it out locally.

@TobiX
Copy link
Owner

TobiX commented Nov 7, 2017

Well, dist is built by Travis-CI after each commit, so should have that fix, and it's working on my own instances... I can also not reproduce the problem in a clean docker container:

  1. Start container:
docker run -p 8080:8080 jenkins/jenkins
  1. Select plugins (None, only pipeline)
  2. Install simple theme plugin
  3. Restart Jenkins
  4. Set theme to https://tobix.github.io/jenkins-neo2-theme/dist/neo-light.css
  5. Create new pipeline project

Editor works...

Maybe there is anything else breaking the style for the ACe editor?

@slide
Copy link

slide commented Nov 8, 2017

The editor can be typed in, but the cursor is offset from where the typing actually happens. I'll try with the docker as you have and verify.

@TobiX
Copy link
Owner

TobiX commented Nov 8, 2017

Yes, I know how the bug manifests, but I haven't seen it in my installations since I applied ab1d9d0... At least on Windows 10 or Linux with Firefox... I just noticed the bug still occurs in Chrome, sorry. I'll look for a better fix.

@TobiX TobiX closed this as completed in 954bbc0 Nov 8, 2017
@slide
Copy link

slide commented Nov 8, 2017

Thanks for looking into this more, it's a really nice theme and I definitely appreciate your work!

@johnarnold
Copy link
Author

Fixed, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants