-
Notifications
You must be signed in to change notification settings - Fork 92
improve hints on splash; fix quick options saving #791
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
base: master
Are you sure you want to change the base?
Conversation
hendriktews
commented
Oct 2, 2024
- add menu entry to disable splash screen to quick options
- add info to the splash screen on how to leave and disable the splash screen
- add Proof-General menu to splash screen, such that the hints on the splash screen make sense
- fix saving of quick options for several options
- the CHANGES entry also describes commit ea0f007
- adapt manual
- add menu entry to disable splash screen to quick options - add info to the splash screen on how to leave and disable the splash screen - add Proof-General menu to splash screen, such that the hints on the splash screen make sense - fix saving of quick options for several options - the CHANGES entry also describes commit ea0f007 - adapt manual
@Matafou @erikmd : With the information about how to disable the splash screen present on the splash screen, I think we can and also should increase the splash screen time again, maybe 3 or 4 seconds. With 1 second, it is impossible to see that there is information about how to disable it. What do you think? |
IMHO, I would not come back to a 3--4 seconds delay: I would just disable the splash feature by default, or ideally (to be less "aggressive") modify the splash feature so that the *Proof General Welcome* buffer is still created, but in the background (i.e. similarly to calling |
FWIW, I'd vote to remove the splash screen.
I think it makes sense to display such a thing when the users ask "About
Proof-General" or in response to a `M-x proof-general` command (maybe
from which the user could choose different backends or some such), but
doing it when opening a `.v` file is more annoying (and difficult to
implement robustly) than it's worth.
For the record, I disabled it the first time I tried to use PG because
it caused an error in my setup, probably due to either my
minibuffer-only frame or my dedicated windows (I never bothered to track
it down nor try and see if it had been fixed in the mean time).
|
Agreed. A "about" command would be better. |
Hi, To sum up, no one objected to "improving" the splash screen. Also, it was suggested to simply remove it. So, if no one objects, I will merge #791 then #793 at the end of the week. |
Fix commit 1f9484f by removing the CHANGES note that belongs to PR ProofGeneral#791.
@Matafou approved this PR #791; @hendriktews do you also agree? or you would actually favor a more abrupt solution, |