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

[OSK] Implement the welcome warning box on startup #1007

Merged
merged 1 commit into from
Nov 17, 2018

Conversation

GeoB99
Copy link
Member

@GeoB99 GeoB99 commented Nov 2, 2018

Purpose

In Windows XP, both Magnify and On-Screen Keyboard show to the screen a welcome dialog box informing the user about the purpose of those applications. This feature is already implemented for Magnify in ReactOS which however in On-Screen Keyboard is still unimplemented.

Presentation

URL Video: https://www.youtube.com/watch?v=daJdvnI5tiI

Proposed Changes

  • Implement LoadDataFromRegistry() and SaveDataToRegistry()
  • Implement OSK_WarningProc() which is used for the handling of the dialog box
  • Implement localization support

base/applications/osk/main.c Outdated Show resolved Hide resolved
base/applications/osk/main.c Outdated Show resolved Hide resolved
base/applications/osk/main.c Outdated Show resolved Hide resolved
base/applications/osk/main.c Show resolved Hide resolved
base/applications/osk/settings.c Outdated Show resolved Hide resolved
base/applications/osk/main.c Outdated Show resolved Hide resolved
base/applications/osk/main.c Show resolved Hide resolved
Copy link
Contributor

@katahiromz katahiromz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs some changes...

@GeoB99 GeoB99 force-pushed the osk-showwarning branch 2 times, most recently from e2eaddd to 05ee917 Compare November 2, 2018 17:06
base/applications/osk/settings.c Outdated Show resolved Hide resolved
base/applications/osk/settings.c Show resolved Hide resolved
base/applications/osk/settings.c Show resolved Hide resolved
base/applications/osk/settings.c Outdated Show resolved Hide resolved
base/applications/osk/settings.h Outdated Show resolved Hide resolved
base/applications/osk/settings.h Outdated Show resolved Hide resolved
base/applications/osk/settings.c Outdated Show resolved Hide resolved
@GeoB99 GeoB99 force-pushed the osk-showwarning branch 3 times, most recently from 8d9d41a to 9203737 Compare November 3, 2018 13:39
base/applications/osk/settings.c Outdated Show resolved Hide resolved
base/applications/osk/settings.c Outdated Show resolved Hide resolved
base/applications/osk/settings.c Outdated Show resolved Hide resolved
@GeoB99 GeoB99 force-pushed the osk-showwarning branch 2 times, most recently from 7145047 to 59753e6 Compare November 10, 2018 14:59
base/applications/osk/settings.h Outdated Show resolved Hide resolved
@GeoB99
Copy link
Member Author

GeoB99 commented Nov 17, 2018

@HBelusca Review addressed.

In Windows XP, both in On-Screen Keyboard and in Magnify there's a welcome box which is shown upon startup of the application. Our Magnify already has this thing implemented meanwhile in On-Screen Keyboard it's not.
@HBelusca HBelusca merged commit 34c8123 into reactos:master Nov 17, 2018
@GeoB99 GeoB99 deleted the osk-showwarning branch November 17, 2018 20:31
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

Successfully merging this pull request may close these issues.

5 participants