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

Grid pattern support in editor area #1704

Closed
arigit opened this issue Dec 2, 2017 · 6 comments
Closed

Grid pattern support in editor area #1704

arigit opened this issue Dec 2, 2017 · 6 comments

Comments

@arigit
Copy link

arigit commented Dec 2, 2017

Feature request: "Display Grid Pattern" support.
Both gEdit (optional) and gnome-Builder (by default) support displaying a subtle grid pattern in the editor area.

The pictures below show how the grid looks like with different solarized theme flavors. I understand this is a matter of taste but since this became Builder's default I guess this is popular with devs; I don't use builder but I find myself using it in gEdit all the time and I miss it in Geany.

I couldn't find any open or closed feature requests on geany so here it goes. Some research shows this was a feature added by gtksourceview.

image

image

@elextr
Copy link
Member

elextr commented Dec 2, 2017

I think this would need to be implemented by Scintilla, the editing widget Geany uses which is a separate project here. Then Geany would then only need an option to turn it on (off by default).

@b4n
Copy link
Member

b4n commented Dec 4, 2017

I'm still puzzled one what is the purpose of this pattern, esp. as on your screenshots the grid doesn't even seem to match line boundaries or something.

But anyway, as @elextr mentioned, if Scintilla gets support for it we could add a setting easily (or a plugin could add it, even). And implementing this without modifying Scintilla itself, while I believe would be theoretically possible, would likely be a pain and a lot less efficient -- and in any case not worth the hassle IMO.

@elextr
Copy link
Member

elextr commented Dec 4, 2017

while I believe would be theoretically possible, would likely be a pain and a lot less efficient -- and in any case not worth the hassle IMO.

Well, the last attempt to modify the background had to be removed after a while IIRC.

@b4n
Copy link
Member

b4n commented Dec 4, 2017

@elextr wasn't it VTE background that was removed because newer versions of the VTE lib dropped the feature, and dealing with it manually was too much trouble?

@elextr
Copy link
Member

elextr commented Dec 4, 2017

Yes, might have been VTE.

@kugel-
Copy link
Member

kugel- commented Jan 30, 2018

I'm with @b4n here. It doesn't seem useful if it doesn't even align to line heights / characters. And it would be a lot of work, and needs to be implemented in scintilla (we like to avoid invasive local changes to it).

@kugel- kugel- closed this as completed Jan 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants