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

Wrapped text shifts left when company auto-complete comes up #53

Closed
codygman opened this issue Aug 24, 2020 · 8 comments
Closed

Wrapped text shifts left when company auto-complete comes up #53

codygman opened this issue Aug 24, 2020 · 8 comments

Comments

@codygman
Copy link

emacs: 28.0.50
olivetti version:

;; Package-Version: 20200702.601
;; Version: 1.11.1

Given text like this and a configuration where olivetti wraps the line:

complete after this: wo_ <---- cursor here, then trigger completion window
* This is a very very very long line that is longer than my olivetti mode keeps on one line and wraps to the second

That second line will shift to the left and it's very visually distracting lately, though I could easily ignore it at first.

Related:

company-mode/company-mode#272
zk-phi/sublimity#21

@rnkn
Copy link
Owner

rnkn commented Aug 24, 2020

Can't reproduce this sorry...
Screen Shot 2020-08-24 at 4 34 38 pm

@dgutov
Copy link

dgutov commented Aug 24, 2020

Perhaps you have to use the same version of company to reproduce (most recent, from MELPA, right?).

@rnkn
Copy link
Owner

rnkn commented Aug 24, 2020

Then that would indicate a regression in company-mode.

@rnkn rnkn closed this as completed Aug 24, 2020
@codygman
Copy link
Author

codygman commented Aug 24, 2020

I'll try to post exact versions including other package versions and a screencast gif with repro later.

@rnkn
Copy link
Owner

rnkn commented Aug 25, 2020

The other possibility is that someone broke the window display code in Emacs again, given that this happened three times in the last four major versions...

If you can provide a backtrace that would be more informative than versions. Maybe also have a look in your NEWS file for relevant changes

@rnkn rnkn reopened this Aug 25, 2020
@dgutov
Copy link

dgutov commented Aug 25, 2020

I haven't tried olivetti yet, but the relevant change in Company might be the switch to using after-string instead of display overlay property for popup rendering. Which will be needed to show icons in the popup.

Perhaps it interferes with stuff this mode does?

@rnkn
Copy link
Owner

rnkn commented Aug 26, 2020 via email

@dgutov
Copy link

dgutov commented Aug 26, 2020

All right, thank you for trying.

Let's wait for @codygman with more details, then.

@rnkn rnkn closed this as completed Sep 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants