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

[themes] Make folding indicator theme-able #172881

Closed
mtbaqer opened this issue Jan 31, 2023 · 11 comments · May be fixed by #173203
Closed

[themes] Make folding indicator theme-able #172881

mtbaqer opened this issue Jan 31, 2023 · 11 comments · May be fixed by #173203
Assignees
Labels
feature-request Request for new features or functionality themes Color theme issues

Comments

@mtbaqer
Copy link

mtbaqer commented Jan 31, 2023

I'm currently working on the Better Folding extension, and would love to have the ability to customize the folding indicator's color.

My use case is simply setting the color's alpha to 0 to hide it, and show the customized collapsed text provided by the extension.

I am aware of #170447 (I am the author), but until that is merged, I could use this.

Would be more than happy to make the PR for this, just wanted to make an issue to see if there is any limitations I should know first.

@aeschli
Copy link
Contributor

aeschli commented Jan 31, 2023

We have a setting to hide the folding twisty next to the line numbers. Is that helpful?

@aeschli aeschli added the info-needed Issue requires more information from poster label Jan 31, 2023
@mtbaqer
Copy link
Author

mtbaqer commented Jan 31, 2023

Apologies for not being clear. By folding indicator I meant current placeholder for the collapsed text. The "..." that shows at the end of the folded line.

@aeschli
Copy link
Contributor

aeschli commented Feb 1, 2023

Ok, I see. It's a bit of a workaround to add a color that can then will be made transparent. Especially as control of colors comes from themes. We're still waiting for @alexdima to be back and discuss #170447.

@aeschli aeschli changed the title Make folding indicator theme-able [themes] Make folding indicator theme-able Feb 1, 2023
@aeschli aeschli added themes Color theme issues feature-request Request for new features or functionality and removed info-needed Issue requires more information from poster labels Feb 1, 2023
@aeschli aeschli added this to the Backlog Candidates milestone Feb 1, 2023
@VSCodeTriageBot
Copy link
Collaborator

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

@mtbaqer
Copy link
Author

mtbaqer commented Feb 1, 2023

I admit that it is definitely a workaround (just like many things with the extension). It is just that changes of that size usually take a bit of time to be approved and merged, while adding this should be relatively simple (again, more than happy to provide a PR) and would go long way for the extension.

It also comes with the benefit of providing one more thing users can personalize as well.

@aeschli
Copy link
Contributor

aeschli commented Feb 3, 2023

The problem is really that colors come from themes. Your extension can not / should not overwrite that color.

@mtbaqer
Copy link
Author

mtbaqer commented Feb 6, 2023

I agree 100% that my extension should not overwrite this. However, I would like to make the cases of:

Don't get me wrong, I understand that providing these features takes a lot of time and effort while at the same time joggling the rest of the backlog, and you guys doing more than an amazing job at it. All I'm asking is to allow us the extension authors to fill in the gaps in the meantime, and sometimes the only way we can deliver these requested features is using these hacky solutions. Of course they don't work for many cases, but a good enough hacky solution is much better than no solution at all.

This was the long way of me saying: yes, my extension should not do this, but until #170447 gets merged, this might be our best alternative for now.

@aeschli
Copy link
Contributor

aeschli commented Feb 14, 2023

So if we added that color, how would your extension set it to transparent?

@mtbaqer
Copy link
Author

mtbaqer commented Feb 14, 2023

My initial thoughts are changing the workplace settings (like how vscode-peacock does it) to change the color to something with alpha value 0 (for example #00000000). Maybe even allow an option to make it apply to universal settings as well.

I'm also thinking of having it apply on activate() and only on supported languages, and be removed on dispose() and while using unsupported languages.

Of course all while making sure that users are aware of this and why we are doing it with an option to deactivate it and fallback to the current solution (with all its downfalls).

@VSCodeTriageBot
Copy link
Collaborator

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

@VSCodeTriageBot
Copy link
Collaborator

🙁 In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

@VSCodeTriageBot VSCodeTriageBot closed this as not planned Won't fix, can't repro, duplicate, stale Apr 4, 2023
@github-actions github-actions bot locked and limited conversation to collaborators May 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality themes Color theme issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants