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

version 1.01 fixed and improved for Delphi 12. #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SkybuckFlying
Copy link

  • IFNDEFS problems with FPC solved for Delphi.
  • Position set to screen center.
  • Controls widened a bit to be able to input/see/handle larger security codes.

Application tested and works ! Nice ! =D
Application is compatible with google authenticator.

+ IFNDEFS problems with FPC solved for Delphi.
+ Position set to screen center.
+ Controls widened a bit to be able to input/see/handle larger security codes.

Application tested and works ! Nice ! =D
Application is compatible with google authenticator.
@SkybuckFlying
Copy link
Author

This will make your application a little bit nicer.

@@ -32,7 +36,7 @@ Base32 = class
/// </summary>
/// <returns>
/// Unicode String containing the ANSI-Data from that Base32-Input
/// </returns>
/// </returns>
Copy link
Owner

Choose a reason for hiding this comment

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

This looks unwanted. Please revert

{$ELSE}
, HMAC
{$IFEND};
*)
Copy link
Owner

Choose a reason for hiding this comment

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

Why both ways? If you format the uses clause, please leave only one

@@ -1,5 +1,22 @@
unit Form;

{
Copy link
Owner

Choose a reason for hiding this comment

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

Please remove that comment

@wendelb
Copy link
Owner

wendelb commented Mar 12, 2024

Please also remove the changes to Example. Those seem to be unecessary.

Once that is done, I will merge this PR.

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.

None yet

2 participants