Skip to content

Conversation

@juanotto
Copy link
Contributor

Draft while I try to get some space after the images... when through asciidoctor and antora docs, tried +, {blank}, {empty}, blank lines, lines with spaces...

New versions of the images on the tutorial are added, using the same naming pattern most images follow: _270_ to indicate RELEASE270 was used.

I added 1 new image because the handling of adding new modules is different than in the "Module Suites" era in which the original was written.

Some wording replaced (e.g. Module Suite with Application).

Some images are slightly blurry, better than the previous ones but still... I will try and fix that to actually try to merge this with the 2 parts selection tutorial part1 & part2. In the future :)

@ebarboni
Copy link
Contributor

Hi, thanks for your effort

I do some polish in the wiki, to deal with space and numbered list you should prefers + and not using [start] the numbers should be good by the wiki processor not by you/me hacking.

Extract of a working part of another wiki page

. Modify the ``Main.java`` class to add the following code. Save your changes.
+
[source,java]
----
package demo;
import ann.Handleable;
public class Main extends MainExtras {
@Handleable
private String stuff;
public static void main(String[] args) {
new Main().handleStuff("hello");
}
}
----
+
This code contains the following elements:
+
* import statement for the custom annotation processor ``ann.Handleable``
* the public class ``Main`` that extends the ``MainExtras`` class ( ``MainExtras`` should be generated by the annotation processor during compilation)
* a private field named ``stuff`` that is annotated with the ``@Handleable`` annotation
* the ``main`` method that calls the ``handleStuff`` method, which is declared in the automatically generated ``MainExtras`` class
+
In this simple example, the ``handleStuff`` method only prints out the current value. You can modify this method to perform other tasks.
+
After you save the ``Main.java`` code you will see that the IDE reports multiple compilation errors. This is because the annotation processor has not been added yet to the project.
+
. Right-click the ``Demo`` project node in the Projects window, choose Properties, then select the Libraries category in the Project Properties window.

If you prefer more gap for image, it would be more css related, and should be looked for all tutorials

@juanotto
Copy link
Contributor Author

Thanks for the info! I did some more experimenting and it doesn't seem straightforward :/

The + between list element and the next element keeps the next element inside the <li> but it does not control vertical space in any way. As long as lines are separated by + everything is inside the list element, an empty line drops out of the <li>.

Just to experiment, and make the gods of old hacks happy... I tried {nbsp}! A single <p> can be added this way. I will not put that on a PR though.

I had a look at other pages, I think this needs to be solved for the whole site actually. E.g. form java profiling:
image

I will try to check that at some point but I know I don't have to fix that spacing here.

@juanotto juanotto marked this pull request as ready for review October 14, 2025 00:56
@juanotto
Copy link
Contributor Author

I fixed the format a bit, mostly + to keep elements visually tab'ed, by being inside of <li>s.

@mbien
Copy link
Member

mbien commented Oct 25, 2025

New versions of the images on the tutorial are added, using the same naming pattern most images follow: 270 to indicate RELEASE270 was used.

probably a good choice to add a 0, otherwise it would collide with the other numbers at some point.

@juanotto
Copy link
Contributor Author

New versions of the images on the tutorial are added, using the same naming pattern most images follow: 270 to indicate RELEASE270 was used.

probably a good choice to add a 0, otherwise it would collide with the other numbers at some point.

That's a bit of a stretch buuuut it doesn't hurt, done

@mbien
Copy link
Member

mbien commented Oct 27, 2025

That's a bit of a stretch buuuut it doesn't hurt, done

I am sorry but you misunderstood. You already had the 0 in "270". I only meant to say that I think its ok to use the maven versioning scheme here too, given that we have ancient files in the same folder using "73".

So I think "27" would be fine (possibly better) but "270" is good too since we probably could get rid of the old screenshots at some point.

Super sorry to cause confusion.

@juanotto
Copy link
Contributor Author

That's a bit of a stretch buuuut it doesn't hurt, done

I am sorry but you misunderstood. You already had the 0 in "270". I only meant to say that I think its ok to use the maven versioning scheme here too, given that we have ancient files in the same folder using "73".

So I think "27" would be fine (possibly better) but "270" is good too since we probably could get rid of the old screenshots at some point.

Super sorry to cause confusion.

Now that makes sense! lol no problem, quick fix done. I also checked and all images except one could be safely deleted :) I just pushed those changes

Copy link
Member

@mbien mbien left a comment

Choose a reason for hiding this comment

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

reads good!

@mbien
Copy link
Member

mbien commented Oct 27, 2025

@juanotto if you are happy with this PR, you could add the :page-reviewed: tag and then squash everything into a single commit.

once done we can merge this update

@BradWalker
Copy link
Member

@juanotto , thanks for doing this..

@juanotto
Copy link
Contributor Author

@mbien done, with some hiccups but done... next one will be easier :)

@mbien mbien merged commit 3267081 into apache:main Oct 27, 2025
@mbien
Copy link
Member

mbien commented Oct 27, 2025

is now live at https://netbeans.apache.org/tutorial/main/tutorials/nbm-maven-modulesuite/

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.

4 participants