Discussion :

[keul85]

Bonjour,

J'ai un petit souci avec un GridLayout.

J'ai actuellement une petite application qui crée un JTree en fonction d'un fichier XML. A chaque nœud de cet arbre correspond un dossier d'images. Lorsque je clique sur un nœud, des miniatures de chaque image doivent s'afficher dans un GridLayout. Le problème est que, selon que j'ai 100 images ou seulement 2 à afficher, la taille des miniatures n'est pas là même, ce qui est plutôt très moche.

Ce que j'ajoute au JPanel sont des JLabel, chaque JLabel contenant un ImageIcon. La taille des JLabel étant censé être la même.

Le petit bout de code qui me semble être important :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
        //Le panel contenant les miniatures est contenu dans un JScrollPane si le nombre de miniatures à afficher est trop important
JScrollPane jsp = new JScrollPane(panelMiniaturesPhotos, JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED, JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
 
 
	/**
         * Methode qui modifie le GridLayout en fonction du nombre de lignes à afficher.
         * @param numberOfPhotos Le nombre de photos à afficher
         */
	public void setNewGridLayout(int numberOfPhotos) {
		panelMiniaturesPhotos.removeAll();
		GridLayout gl = (GridLayout) panelMiniaturesPhotos.getLayout();
		gl.setColumns(3);
		gl.setHgap(10);
		gl.setVgap(15);
		gl.setRows(((Double)Math.ceil(numberOfPhotos/3)).intValue()+1);
		panelMiniaturesPhotos.setLayout(gl);
	}
 
	/**
         * Methode qui ajoute une photo au panel
         * @param photoSelected le path vers la photo à ajouter
         */
	public void addPhotoMiniature(String path) {
		ImageIcon photoSelected = new ImageIcon(path);
		JLabel pp = new JLabel(photoSelected);
		pp.setPreferredSize(new Dimension(100,70));
		pp.repaint();
		panelMiniaturesPhotos.add(pp);
		panelMiniaturesPhotos.repaint();
	}

J'ai un peu du mal à comprendre pourquoi les miniatures n'ont pas la même hauteur selon le nombre à afficher.

Si quelqu'un a une idée, je la prends avec plaisir.

Merci d'avance.

[gifffftane]

Comme il est indiqué ici et là, le GridLayout divise l'espace disponible en cellules de tailles égales, et donc ne tient aucun compte de la preferredSize.

Solutions possibles :

• caler toujours le GridLayout au nombre maximal d'images possible
• utiliser un SpringLayout (guide ici), plus difficile à manier, mais plus amical avec la preferred

[keul85]

Merci gifffftane pour la réponse.

Je vais essayer de m'en sortir avec les SpringLayout, qui me semblent être la meilleure solution. Je vous fais signe dès que ça marchera nickel... autant dire que ca peut prendre un peu de temps

[keul85]

Bon, après test, je suis plutôt satisafait du SpringLayout, puisque mes images font maintenant toutes la même dimension, ce que je voulais au départ. Cependant, un nouveau problème m'est apparu, problème que je n'arrive pas à résoudre...

En effet, avant d'utiliser le SpringLayout, mes images étaient certes redimensionnées, mais il était possible de les faire défiler lorsqu'il y en avait beaucoup.

Désormais, avec le SpringLayout, je ne vois que les premières images, et je n'ai plus de scrollbar, ce qui est plutôt très gênant en fait.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
//Dans le constructeur de l'Interface
 
panelMiniaturesPhotos = new JPanel(new SpringLayout());
JScrollPane jsp = new JScrollPane(panelMiniaturesPhotos);
tabInfos.addTab("Miniatures photos", jsp);

L'ajout des photos se fait dans la méthode suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
public void addPhotoMiniatures(LinkedList<String> listOfPhotosInsideDirectory) {
		panelMiniaturesPhotos.removeAll();
		Iterator<String> i = listOfPhotosInsideDirectory.iterator();
		String path;
		JLabel tmp1 = null, tmp2 = null, tmp3 = null;
		double number = 0;
		while(i.hasNext()) {
			path = i.next();
			ImageIcon photoSelected = new ImageIcon(path);
			JLabel pp = new JLabel(photoSelected);
			pp.setPreferredSize(new Dimension(190,120));
			pp.repaint();
			pp.addMouseListener(new OneExifPhotoListener(this, path, pp));
			panelMiniaturesPhotos.add(pp);
			if(number < 3) { //Pour les 3 premières images
				if(number == 0) { //Si c'est la première photo, on la place en haut à gauche
					((SpringLayout)panelMiniaturesPhotos.getLayout()).putConstraint(SpringLayout.WEST, pp, 5, SpringLayout.WEST, jsp);
				} else { //Sinon, pour les 2eme et 3eme photos, on les place horizontalement à 5 unités de la précédente
					((SpringLayout)panelMiniaturesPhotos.getLayout()).putConstraint(SpringLayout.WEST, pp, 5, SpringLayout.EAST, tmp1);
				} //La nouvelle photo est placé à 5 unités du bord supérieur du layout
				((SpringLayout)panelMiniaturesPhotos.getLayout()).putConstraint(SpringLayout.NORTH, pp, 5, SpringLayout.NORTH, jsp);
			} else { //Pour les autres
				if(number/3 - Math.round(number/3) == 0) { //Si elle est dans la premiere colonne (multiple de 3), on crée des contraintes pour que la photo soit a 5 unités du bord du layout (horizontalement)
					((SpringLayout)panelMiniaturesPhotos.getLayout()).putConstraint(SpringLayout.WEST, pp, 5, SpringLayout.WEST, jsp);
				} else { //Sinon, elle est placée horizontalement à 5 unité de la photo précédente
					((SpringLayout)panelMiniaturesPhotos.getLayout()).putConstraint(SpringLayout.WEST, pp, 5, SpringLayout.EAST, tmp1);
				} //La nouvelle photo est placée 5 unités en dessous de celle placée en avant-avant-dernier.
				((SpringLayout)panelMiniaturesPhotos.getLayout()).putConstraint(SpringLayout.NORTH, pp, 5, SpringLayout.SOUTH, tmp3);
			}
 
			tmp3 = tmp2;
			tmp2 = tmp1;
			tmp1 = pp;
			number++;
		}
		panelMiniaturesPhotos.repaint();
	}

Est-ce que SpringLayout et JScrollPane sont incompatibles où, ce qui me semble plus vraisemblable, je ne les utilise pas bien ?

Merci d'avance à tous ceux qui se pencheront sur mon cas.

[gifffftane]

Oui mais avant, au lieu de faire...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
JScrollPane jsp = new JScrollPane(panelMiniaturesPhotos);

... tu faisais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
JScrollPane jsp = new JScrollPane(
  panelMiniaturesPhotos, 
  JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED, 
  JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);

Hum.

[keul85]

En effet, mais cela ne change rien. Que je mette l'un ou l'autre, toujours pas de scrollbar.

[gifffftane]

As-tu appliqué Dynamically Changing the Client's Size de How to Use Scroll Panes ?

[keul85]

Ahhhh !!!

Je faisais bien le revalidate() et le repaint(), mais pas le setPreferredSize()...

Je te remercie beaucoup gifffftane !!