Closed (won't fix)
Project:
Image
Version:
6.x-1.x-dev
Component:
image.module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
19 Jul 2005 at 10:00 UTC
Updated:
18 May 2007 at 09:49 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
boris mann commentedCan you please attach a screenshot and describe the workflow.
I am likely going to -1 anything that adds code to image in core, unless it is super fantastic, especially when calling solutions like img_assist complex (a great, useful, and easy module in my opinion).
Comment #2
Bèr Kessels commentedsee my much simple approach -for core- at http://drupal.org/node/26288
Comment #3
jose reyero commentedBer, I like your approach too. Actually I think that upload module could be extended a little bit.
But, is there any reason why we cant have both? I see them as complementary, more than a replacement.
Comment #4
Bèr Kessels commentedWell. In general I am against any options for visual things. Layout must be done in the theme. Not with options. So a definite -1 on the placement options.
I like the idea though, bu why can we not do this the simple way: wtih a contrib that handles [image:] tokens?
Comment #5
jose reyero commentedOk, your objection -for the positioning- makes a lot of sense, so I could have saved that part. I was just thinking of some module that you just enable and then, voila!, you have images in stories. I'll try to rework this a little bit then.
But about [images: ] tokens, that is ok for content editors, but way too complex for too many users. I need simple, simple, simple image handling...
However, my ideal solution would be adding some 'type' field to the files table, and then define some values like 'attachment', 'thumbnail', 'picture', etc.... this way, the image.module wouldn't have to mess with file names and this objects could be used accross Drupal by any module and theme....
But, when it comes to the interface for adding images to stories, blogs, etc, I think this is really the best. It's not a replacement, but fully compatible with that [images: ] that can be handled by more skilled editors.
Comment #6
jose reyero commentedI've just removed all that settings, and replaced them with a themeable function.
Comment #7
venkat-rk commentedAbsolutely. This should how it should be to the end users. If the intent of this patch was to make it easy for users to define placement of images, then the removal of placement settings is a step backward.
-1 from me as a simple user. Why should the placement of images be left *only* to the themers? Can we at least have the placement settings as an option in addition to the themeable function?
Comment #8
Bèr Kessels commentedBefore you say tghat you need placement of images you should have a look on the internet.
ou will see that *hardly any* good CMS diven site allows placement of images. There are loads of reasons, but it comes down to: added complexity for something ppl dont want.
I developed an ilmage placement thing for drupal, for a client (cost a lot of money) and gues what? They used it to place the image on exact the same place on each page! Really, the EASY solution is a default placement, coded in the theme;
Power users should use HTML and something like a wysiwyg or img assist.
Comment #9
Anonymous (not verified) commentedHere's an update of Jose's patch for 4.7. I think this is still the easiest way for users to display an image with a node, but please tell me if I've missed something! Jose, thanks for sharing this in the first place.
Comment #10
guardian commentedIn Drupal 5.x, you can use either http://drupal.org/project/upload_image which takes uploaded images, that are attached to nodes, and creates image nodes from them. It also displays thumbnails in place of the file name in the list of attachements.
You could also use CCK, and particularly the http://drupal.org/project/imagefield field to enable single or multiple image fields in your node type.
Comment #11
guardian commented