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

Separate FLUID user documentation, screen shot automation #936

Merged
merged 54 commits into from Apr 17, 2024

Conversation

MatthiasWM
Copy link
Contributor

Incrementally adding user documentation for FLUID. Non-widget node types are don at this point.

I am tying to user automatically generated screen shots to document FLUID features visually. To not pollute the repo with hundreds of small png's, screen shots are created by FLUID with a new command line option --autodoc path.

I am not great at writing documentation, neither in English nor in German, so the handbook will be more of a pice by piece description of what elements in the user interface do. But that's probably still better than no docs at all. If all goes well, I will move the FLUID parts of the FLTK docs over here.

For now, this is more of a pull announcement than a pull request. I am always happy to hear recommendations and fixes.

@MatthiasWM MatthiasWM self-assigned this Mar 17, 2024
@MatthiasWM MatthiasWM added fluid Fast Light User Interface Designer (fluid) documentation Needs better documentation active Somebody is working on it labels Mar 17, 2024
@MatthiasWM
Copy link
Contributor Author

State of the FLUID Docs: http://elektriktrick.com/fluid/index.html

@Albrecht-S
Copy link
Member

Great stuff for users and developers like me. I can't tell much about the contents (I don't know fluid well) and I didn't have the time for full proof reading. Would it make sense to check text and typos at this time, or are you still developing (and/or rewriting) substantially?

@MatthiasWM
Copy link
Contributor Author

@Albrecht-S Thanks for taking quick look. As a next iteration, I will merge the existing documentation with what I wrote here and create a hopefully helpful and complete handbook. I will post here when that is done. That's probably the best time for proof reading.

If devs then agree, I will remove the current FLUID docks from the FLTK docs and just leave a reference to advertise the existence of FLUID there. After that, the pull request can be applied in the hopes that 1.4.0 gets some shiny new detailed docs for FLUID.

I purposely do not include inner workings or a class reference. Not everything is documented anyway, and 1.5 will bring a new constant naming scheme.

@MatthiasWM
Copy link
Contributor Author

Ok, so I am 98% done. I merged the original docs with the new docs and fixed what was changed in FLUID. I still want to add two short tutorials to bring some points across. Other than that, happy to hear comments.

Also, we need to decide on how to reference the new docs from the FLTK docs, and If/where we want to put them on the fltk.org web page.

Refreshed http://elektriktrick.com/fluid/index.html .

@MatthiasWM MatthiasWM changed the title InProgress: FLUID user documentation, screen shot automation Separate FLUID user documentation, screen shot automation Apr 11, 2024
@MatthiasWM MatthiasWM marked this pull request as ready for review April 11, 2024 19:13
sudo apt-get install -y doxygen=1.10.0
sudo apt-get install texlive-latex-base
sudo apt-get install texlive-fonts-recommended
sudo apt-get install texlive-latex-extra
Copy link
Member

Choose a reason for hiding this comment

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

@MatthiasWM Instead of all these texlive-* packages installing doxygen-latex should do it. This should also imply doxygen but I don't know if doxygen=1.10.0 works (?)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope, Ubuntu is at 1.9.2. 1.10.0 does not work :-(


add_custom_target(fluid_docs
COMMAND echo fluid::fluid -scheme gtk+ --autodoc ${CMAKE_CURRENT_BINARY_DIR}/src/
COMMAND fluid::fluid -scheme gtk+ --autodoc ${CMAKE_CURRENT_BINARY_DIR}/src/
Copy link
Member

Choose a reason for hiding this comment

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

s/fluid::fluid/fltk::fluid/ ???

Copy link
Contributor Author

Choose a reason for hiding this comment

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

LOL, yes. Sorry, I will generate a bunch of commits until the workflow is well.

Copy link
Member

Choose a reason for hiding this comment

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

Sorry, I will generate a bunch of commits until the workflow is well.

That's what forks are good for. ;-)

@MatthiasWM MatthiasWM marked this pull request as draft April 14, 2024 19:03
@MatthiasWM
Copy link
Contributor Author

PDF no longer shows front logo

To move or resize a widget, it must be selected first by clicking on it.
Multiple widgets can be selected by holding down the Shift key when clicking
on them, or by dragging a selection box around widgets. Widgets can also be
selected in the widget browser the main window. Shift-click will select a
Copy link
Member

Choose a reason for hiding this comment

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

in the widget browser the main window

should read: "in the widget browser {in|of} the main window"

may not follow changes in the grid widgets. It's best to first complete the
grid, and the add children to the grid cells later, size them correctly, and
may not follow changes in the grid widgets. It's best to complete the grid
first, then add children to the grid cells later, size them correctly, and
the finally lay out the grid cell children.
Copy link
Member

Choose a reason for hiding this comment

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

IMHO either then finally or just finally rather than the finally

__Edit > Show or Hide Source Code (Shift+Alt+S)__: This button shows or hides
the source code preview window. Any change to the layout or code in the layout
editors can be previewed and verified immediately in the Code View window.
__Edit > Show or Hide Source Code (Shift+Alt+S)__: Shows or hide
Copy link
Member

Choose a reason for hiding this comment

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

s/hide/hides/

@@ -20,15 +20,15 @@
a dialog box will pop up, explaining what type of parent node is required.

The Window and Widget Class icons can be dragged onto the desktop,
creating a new window at the drop position.
creating a new window or widgetat the drop position.
Copy link
Member

Choose a reason for hiding this comment

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

widgetat?

@MatthiasWM MatthiasWM marked this pull request as ready for review April 17, 2024 15:46
@MatthiasWM MatthiasWM merged commit fd791a0 into fltk:master Apr 17, 2024
@Albrecht-S Albrecht-S added fixed The issue or PR was fixed. and removed active Somebody is working on it labels Apr 17, 2024
@Albrecht-S
Copy link
Member

Thanks very much, great job. I built the Fluid docs successfully on Linux, and installation worked as well as expected.

Awesome!

One really tiny issue though:

$ ninja html
[2/2] Generating HTML documentation
/git/fltk/master/documentation/src/forms.dox:21: warning: unable to resolve reference to 'fluid' for \ref command

@MatthiasWM
Copy link
Contributor Author

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Needs better documentation fixed The issue or PR was fixed. fluid Fast Light User Interface Designer (fluid)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants