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

Text view issue in A6.1 alpha version #255

Open
grayswandyr opened this issue Apr 11, 2024 · 6 comments
Open

Text view issue in A6.1 alpha version #255

grayswandyr opened this issue Apr 11, 2024 · 6 comments

Comments

@grayswandyr
Copy link
Contributor

Given this model:

sig S {}
run { all w,x,y,z: S | some u: S, disj s1, s2 : set S | #s1 + #s2 > 3 } for 4

Alloy 6 gives the following text view (after hitting New once):
image

while the Alloy 6.1 alpha version (seen in commit d531233) gives:
image

@grayswandyr grayswandyr added this to the A6.1 milestone Apr 11, 2024
@grayswandyr
Copy link
Contributor Author

PS: a Skolem depth of 4 must be set to test this.

@nmacedo
Copy link
Contributor

nmacedo commented Apr 11, 2024

Don't know what caused the change, but for the record it looks fine in macos.

Screenshot 2024-04-11 at 16 40 37

@grayswandyr
Copy link
Contributor Author

Clearly, one difference is between both my versions is apparently the switch from a vertical scrolling to an horizontal one. Might be buggy on Linux. Can we switch back to vertical (and will it be compatible with the great diff highlighting you implemented) ?

@nmacedo
Copy link
Contributor

nmacedo commented May 3, 2024

It's just a matter of switching the wrap flag to true here, but I don't know why it was changed in the first place, probably because wrapping messes us the table view?

final JTextArea ta = OurUtil.textarea(text, 10, 10, false, false);

@pkriens
Copy link
Contributor

pkriens commented May 14, 2024

the horizontal scrolling was added for the table view, yes.

I will make it wrapped for text and unwrapped for tables.

@pkriens pkriens closed this as completed May 14, 2024
@grayswandyr
Copy link
Contributor Author

This is still broken.
Screenshot from 2024-09-26 17-18-35

@grayswandyr grayswandyr reopened this Sep 26, 2024
nmacedo added a commit to nmacedo/org.alloytools.alloy that referenced this issue Oct 16, 2024
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

3 participants