Skip to content

Conversation

@mukeshdhadhariya
Copy link
Contributor

This PR resolves issue #22

  1. CTkLabel Image Warning

    • Problem: Passing PIL.ImageTk.PhotoImage to CTkLabel triggers a warning:
      "Given image is not CTkImage but <class 'PIL.ImageTk.PhotoImage'>."
    • Cause: CTkLabel expects a CTkImage for proper rendering and high-DPI support.
    • Fix: Replaced ImageTk.PhotoImage with customtkinter.CTkImage, ensuring proper scaling and eliminating the warning.
  2. TclError for CTkEntry State

    • Problem: Accessing entry state via receiver_entry['state'] raises _tkinter.TclError: unknown option "-state".
    • Cause: CTkEntry does not support dictionary-style access for widget options like normal Tkinter widgets.
    • Fix: Replaced receiver_entry['state'] with receiver_entry.cget("state") for reading and receiver_entry.configure(state="...") for writing. This aligns with CustomTkinter's API and prevents runtime errors.

@mukeshdhadhariya
Copy link
Contributor Author

@AdityaKumar5155 Sir, review this PR which solve issue #22

@AdityaKumar5155
Copy link
Collaborator

I have reviewed your changes :-

  1. The target change you performed is working as i observed.
  2. The logo size is increased than that of original version. So, i would like if you resized the image similar to original version.
  3. Your commit message is not following OPCODE guidelines, if you are unaware of that you can contact me.

Due to 3rd i am closing this PR but your contribution will be accepted so feel free to create a new PR with the guidelines.

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.

2 participants