[ADDON][WIP] In Line Attachments

Topic: [ADDON][WIP] In Line Attachments (Read 21892 times) previous topic - next topic

0 Members and 1 Guest are viewing this topic.

Re: [ADDON][WIP] In Line Attachments

Reply #15 – November 06, 2014, 04:09:53 pm

but it topic view I can see the image:

skodaclub.it/skodaelk10/

Re: [ADDON][WIP] In Line Attachments

Reply #16 – November 06, 2014, 04:57:41 pm

Now I remember that bug from SMF days, its due to the $topic not being set when coming in from a portal page. If you look at the links on those images they say topic=0.0 I'll get that fixed in a bit.

Re: [ADDON][WIP] In Line Attachments

Reply #17 – November 17, 2014, 12:44:19 am

sorry if I insist, but was this solved? thank you in advance

Re: [ADDON][WIP] In Line Attachments

Reply #18 – November 17, 2014, 01:48:25 am

Not resolved but not forgotten either ... will get it fixed soon, I promise !

Re: [ADDON][WIP] In Line Attachments

Reply #19 – November 18, 2014, 12:18:37 am

Please replace your ILA.subs.php with this one and let me know if it resolves the issue.

Re: [ADDON][WIP] In Line Attachments

Reply #20 – November 18, 2014, 01:10:39 am

yes, the inline images are showed correctly on portal

there is another problem with the titles:

Code: [Select]

skodaclub.it/skodaelk10/index.php

Re: [ADDON][WIP] In Line Attachments

Reply #21 – November 18, 2014, 02:07:09 am

Thats the floats acting up ... you can try adding

Code: [Select]

.sp_article_content .righttext {
	clear: both;
}

to portal.css ... after the .sp_article_content selector. May be a better way but that should work.

Re: [ADDON][WIP] In Line Attachments

Reply #22 – November 18, 2014, 02:28:59 am

that worked, thanks!

Re: [ADDON][WIP] In Line Attachments

Reply #23 – November 24, 2014, 05:53:02 pm

In the admin settings...what does "(ILA) Only show basic ILA menu" do?

Re: [ADDON][WIP] In Line Attachments

Reply #24 – November 24, 2014, 06:05:13 pm

Quote from: b4pjoe - November 24, 2014, 05:53:02 pm

In the admin settings...what does "(ILA) Only show basic ILA menu" do?

for now nothing, in the old version for SMF it was showing a dropdown menu from where you could set the position. Once set the position the mod adds the bbcode. For now ILA for elkarte is WIP (work in progress) and you should write the bbcodes manually.

Re: [ADDON][WIP] In Line Attachments

Reply #25 – November 24, 2014, 06:07:11 pm

Thanks for the info. I thought maybe I was supposed to be seeing something when I checked it...but I wasn't.

Re: [ADDON][WIP] In Line Attachments

Reply #26 – January 02, 2015, 11:43:34 pm

In these days I keep an eye on my error log and I found something regarding the ILA addon: when a user insert a .pdf attacchment and use the code [attachurl=1] I get thison my error log:

Tipo di errore: Generale
Too few arguments: URL http://www.skodaclub.it/index.php?topic=2388.0
File: /....***/sources/subs/ILA.subs.php
Riga: 56

---edit---
I know this addon is wip, but in the future would be nice to have responsive images inserted with [attachimg=x]

Re: [ADDON][WIP] In Line Attachments

Reply #27 – January 07, 2015, 02:51:50 pm

Spuds: in the perspective to bring this into the code (at some point

), what would you think if the attachments are created exactly the moment they are uploaded and passed the tests? Without waiting for the message to be posted?
Thinking quickly about it I would see as pro:
* [attach=1] could become [attach=123] where 123 is the attachment id (that should save some hassles),
* it would allow to simplify the code related to handling attachments in session during posting (we could just store the attach id),
* probably something else
In the con list I would add:
* less "easy" to remove them when not used (well, not exactly but it would be slightly difficult to identify them)
* the id attach could growth slightly faster.

Re: [ADDON][WIP] In Line Attachments

Reply #28 – January 07, 2015, 04:08:30 pm

Quote from: emanuele - January 07, 2015, 02:51:50 pm

in the perspective to bring this into the code (at some point ),

into the core, right?

I like the idea, I imagine this will not happen before 1.1 release

Quote from: emanuele - January 07, 2015, 02:51:50 pm

* [attach=1] could become [attach=123] where 123 is the attachment id

I don't know if this is so good, because you also have to find a solution to convert the existing inline attachments for those who are using this addon. Some softwares like XF are using inline attachments into core and they are using the same bbcodes as elkarte attach=x and attachimg=x (where x is not the attachment id, but the id of attachment into that post) so you will have to do some extra work on open importer.

Re: [ADDON][WIP] In Line Attachments

Reply #29 – January 08, 2015, 01:48:33 am

Thanks for the bug report, I'll take a look. {eta line 56? could you post your subs file, in mine thats a comment line)

RE core ... yeah we have to think about this some for sure, they way it works now is complicated having to do the translation from post ID to real ID. But then there are complications when you go to edit, so you would have to turn it around so it made sense to the poster (meaning =1 or maybe =attachment name) vs =123873 ...

Actually I think its the quote checking that is the most complicated anyway, when I did the mod there was one user who really beat the heck out of it and found all sorts of cool usage cases that I had to work around.

I have not looked at that code since I posted the WIP

I really should !!