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

New layout of “Create class” widget not reflected in docs #6961

Open
kaimikael opened this issue Dec 20, 2024 · 6 comments
Open

New layout of “Create class” widget not reflected in docs #6961

kaimikael opened this issue Dec 20, 2024 · 6 comments
Labels
bug report Bug is reported by user, not yet confirmed by the core team

Comments

@kaimikael
Copy link

What's wrong?

This is just a cosmetic issue, but the “Create class” widget has a new layout which has not been updated in the docs yet, as shown in the attached screenshot.

How can we reproduce the problem?

Drag a “Create class” widget to the canvas, open the widget, press the “?” button, compare the layout of the actual widget with the docs.

What's your environment?

Screenshot 2024-12-20 at 11 36 57
@kaimikael kaimikael added the bug report Bug is reported by user, not yet confirmed by the core team label Dec 20, 2024
@borondics
Copy link
Member

Hi @kaimikael,

Indeed, you are quite right! I started updating docs partially because of your comment. Would you be interested in helping with this? We could agree who takes on what widgets.

@markotoplak or @janezd: is there a guide on writing documentations? I am kind-of familiar with it but others might need help to get started.

@kaimikael
Copy link
Author

Heh, I guess I should help out and not just complain :-) I’m going to be a bit busy the next few weeks, but if you assign me suitable widgets I’ll have a go at them. Then again, I’ll probably have to ask various people about what the unexplained features are, but I guess I should be able to see who’s checked them in.

@borondics
Copy link
Member

borondics commented Jan 5, 2025

If you are OK with splitting by sections that would work for me. I am doing the Data section for now, so you could get started on the Transform section widgets.

As for my workflow: for now I created the black stamps in Inkscape (happy to share) because the stamper tool is no longer working due to some deprecation. I also modified the script for reducing image sizes to be able to work with just one image (biolab/orange-tools#17).

@borondics
Copy link
Member

borondics commented Jan 6, 2025

In the meantime I discovered that there is a wiki on how to write documentation...and also that I wasn't following it. :D

@janezd
Copy link
Contributor

janezd commented Jan 10, 2025

Let's wait for implementation of #6962, which will add another check box. :)

@borondics, if I recall correctly, I've fixed stamper. Does it help if you comment out the following lines?

import sip
sip.setapi("QFileDialog", 2)

and

app.setAttribute(Qt.AA_EnableHighDpiScaling)
app.setAttribute(Qt.AA_UseHighDpiPixmaps)

@borondics
Copy link
Member

Yeah, you are right, it was OK, I wasn't using it correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Bug is reported by user, not yet confirmed by the core team
Projects
None yet
Development

No branches or pull requests

3 participants