http://qs321.pair.com?node_id=366646


in reply to Y.A.N.F: Personal Nodelet enhancements.

Well, I'll repeat some of the feedback you've been given by more than one person that you seem unwilling to listen to.

I see no point in so much code just to restrict what I can do with my personal nodelet. I'd much rather just have a text box. Then I can very easily move things around with cut'n'paste in a WYSIWIG manner (instead of pretending like I'm programming in BASIC and renumbering my lines -- which is an okay interface for web-based reordering in some ways, but doesn't even come close to just being able to move things around with cut'n'paste). I'd like to be able to copy the contents into an editor and sort them or run Perl on them, or whatever. Why make such a complex interface when it mainly restricts what I can do when a very simple (nearly trivial) interface it so much easier to use and more powerful?

I'd rather you rename the node "Personal Nodelet Settings" to match the naming of the other personal settings pages (though there may not be much point in having it as a separate page when it becomes mostly just a single text box).

I guess you didn't bother to look too much at existing data you'd have to deal with before you wrote this code. You said it was "fully tested". Do you bother to test it on any existing personal nodelets? How can you consider that "fully"?

Though such values are outside the scope of the original design of the nodelet so I'm not that concerned that people might have to fix their nodelets to work again (or better).

In fact, I'd just run a quick s/<br>/]<br />\n\[/ on everyone's settings and delete a bunch of code (and test, etc.) and be done. Nearly every aspect of the nodelet now become simple one-liners.

KISS++

- tye        

Update: Added a missing "not".

Replies are listed 'Best First'.
Re^2: Y.A.N.F: Personal Nodelet enhancements. (yuck)
by demerphq (Chancellor) on Jun 14, 2004 at 18:35 UTC

    Hmm. Im trying really hard to remember anyone saying that. Perhaps they or you did one time in the CB or something, but I honestly dont remember. The only comments i really recall were about the Titlebar Settings that I put together. And which i havent applied here. But I dont recall anyone saying anything detailed about the personal nodelet. Beyond a few people saying that they liked it on the test server.

    Regardless. If you idea of a personal nodelet is a scratchpad in a box then I think we can handle that. I will say that my thinking was mostly along the lines of "make the current stuff easier to control" not "redesign it in a totally different fashion".

    If you dont like the name ill change the name. Thats ok too. I have no idea what you talking about with your s/// stuff. But yes if you want to treat the scratchpad as a single document then a lot of things do become simpler.

    I will say one thing. This is the most detailed criticism ive seen of anything ive posted here, including this code about 4 months ago. So i dont feel too bad about it not fitting your exact (unsupplied) specification. In the long run i think most people will appreciate what ive given them even if you don't. And ill also work on giving you your scratchpad in a box too. :-)

    I guess you didn't bother to look too much at existing data you'd have to deal with before you wrote this code.

    Im unclear what you mean. I had a look at a number of peoples $VARS, and i had a detailed look at the code that encoded/decoded them. Also the current code doesnt work the way your s/// suggests. Nor is it lossy to my knowledge. Any data placed inside the text boxes will be stored and retreived faithfully.

    Do you bother to test it on any existing personal nodelets? How can you consider that "fully"?

    Sure i did. Several. Can you detail any errors for me wrt to encoding?


    ---
    demerphq

      First they ignore you, then they laugh at you, then they fight you, then you win.
      -- Gandhi


    • Update:  
    Ok, so people could put anchor tags in before and i didnt realize im sorry. A patch is applied that will a) not screw this up and b) fix it if it has been screwed up automatically.


      The only comments i really recall were about the Titlebar Settings that I put together.

      They were about the design being overkill and about a simple text box being better. They apply to the design of both things. I didn't realize that you weren't "hearing" them to the point that you didn't even understand what we were proposing or I wouldn't have written my reply above in the same way. I even recall the discussion covering how we wouldn't need to filter or anything since they are only displayed to the user who wrote them, so at least I wasn't the only one who understood it. I'm not sure if you hung around for the full duration, though. I'm sorry if this criticism came as a surprise or shock to you. I certainly didn't think it would be.

      If you idea of a personal nodelet is a scratchpad in a box.

      Is that ridicule? (: Well, if a simple textbox interface makes it a scratchpad, then I hope you'll read this scratchpad-in-a-threaded-discussion that I'm writing.

      I will say that my thinking was mostly along the lines of "make the current stuff easier to control" not "redesign it in a totally different fashion".

      Well, I don't see how it is "totally different". But perhaps that is a clue that you should put more effort into thinking about the design, discussing it, and hearing criticisms of it and put less effort into cranking out code. Not to say that I'm not glad you can contrib the effort to write so much code for PerlMonks. I appreciate the work.

      I will say one thing. This is the most detailed criticism ive seen of anything ive posted here, including this code about 4 months ago. So i dont feel too bad about it not fitting your exact (unsupplied) specification.

      Well, I'm sure I've said "just a textbox" to you before. And I don't see how you consider what I wrote to be an "exact specification". About all I've said as far as specification was "just a textbox" and then added some explanation of why I think that is a better idea (which is not 'specification').

      And ill also work on giving you your scratchpad in a box too. :-)

      I have yet to hear any advantage of the complex interface over the simple textbox, so I don't see any reason for "too". This looks like another example of the "I'll do it tye's way just to make him happy" announcement rather that pushing back with some reason why you'd do it differently. I gave reasons why I think a textbox is a better. I didn't just say "I want a textbox". Tell me why your idea is better and you might convince me.

      In the long run i think most people will appreciate what ive given them even if you don't.

      I appreciate the time and effort you've donated. You complain about not getting enough criticism but often don't seem to be able to take it when you do get it (nothing unusual there, of course; taking criticism isn't easy).

      - tye        

        They were about the design being overkill and about a simple text box being better.

        Ok, well i think the only staticly held criticism about either sets of patches was a one liner from belg4mit that says the titlebar settings are too complicated. So if the comments you mention were part of discussing that its quite possible I misunderstood them or to be honest completely ignored them on the grounds that the criticism was singularly uninformed and uninformative that I just felt it to be insulting. (Nobody took any time to really review what the TS stuff was actually doing, nor how the features present there would be implemented in a simpler fashion.) So to put it simply had someone written something like this in the wiki or as a reply to the patch then I would have responded directly. So yes its quite possible the comments you are making now occured during a discussion about TS and that i just blanked it out as uninformed comments.

        Is that ridicule?

        No no. But to me it is a signifigant design difference. Lets consider a couple of subtleties in the PN. Both the original and the relacement are conceived of being a list of link like data, with some subset of N links displayed in the nodelet. So for instance the restriction on a 100 links is iirc in the original code. Now in a freeform text box how do we help the user with this? We cant split the data as its free form. So as they add links the list will grow and grow and grow. Necessating regular visits to cut and paste it down to size etc. Now the code as present has no such problem. Itll only show the number of links the user wants to see (with a hard limit enforced that could be changed if folks want). And itll show either the first or the last N nodes too.

        Anyway, unless you can find where someone wrote that down to me in the wiki or something lets not quibble over whether I knew about the simple textbox idea. Just next time be aware that I _do_ and _will_ react to informative constructive criticism. Also be aware that one liners (and im not saying this only to you) like "I dont see the point" and "its too complex" and "I dont want to use it so why should you put it in" are really not what i consider to be either informative nor constructive. At least certainly not as lone comments.

        Tell me why your idea is better and you might convince me.

        I dont think mixing the idea of a link collector (which is what the PN is in my mind) and a "scratchpad in a nodelet" is a good idea. I think they have different uses, and in some respects are complementary. Above i outline a couple of the reasons. So on that level I dont think there is problem having both. Also, yes my view is that this is very much a "give the lion what he wants for dinner or hear him roar". But that doesnt bother me because i am fairly sure that if youll roar, a bunch of other folks are busy miaowing about the same thing. OTOH, im fairly sure there are lots of folks like me that can see the utility of properly managed list of nodelets and who can also see it being useful to have a free form box as well.

        You complain about not getting enough criticism but often don't seem to be able to take it when you do get it

        Honestly you and I see certain discussions in different lights. I am always happy to hear useful constructive criticism. However like anyone I dont take catty comments like some of the ones i heard about TS very well. Especially when they come from folks who I haven't seen write any non-trivial patches.

        I think the way ive reacted to complaints about my own stuff and other peoples stuff here in the monastery shows that I am responsive to criticism. I had no problem with what the folks said about the NN thing, and i had no problem to putting in the work to extend that to cover as many of the criticisms that I had seen. My response time for responding to nits in my releases, or feature requests or the like should be clear evidence that not only do I accept criticism but i also respond and integrate it as quickly as I reasonably can. But if the thought levels and written criticism that i receive is so shallow that the only permanent record is a comment saying "its too complex" (referring to two seperate patch sets comprising of 1000's of lines of code in probably 10 or so patches) then i don't think im out of line rejecting the criticism as useless.

        If those that want to criticise can't take the time to write a thoughtful paragraph or two about what they think is better until months after the original work is done and applied then I think they need to look at their own critical skills and not at my code. I mean if im willing to hack for hour on end on various aspect of the site to make it better then youd think those with strong feelings about how things go could spare the time to write some more useful comments.

        As an example I point you to the test server where ysth has the new scratchpad stuff metaphorically spread all over the floor. I had a reveiw of the code and to be honest I think its too complicated for what i think hes trying to do. And I said so, in a fairly detailed note about what my aprehensions were, and also admitting that i dont understand the full design objectives so my criticism could be misplaced. I suspect that ysth has or will read it and come back with some explanations or he'll say that my comments made him rethink and that it is in fact too complicated. Either way he got a lot more than a one liner.

        So, to wrap this all up, im happy to take direction in developing the site. Im happy to receive as much constructive friendly well meaning critisim as I can. But im not going to pay any attention to stuff that isnt such. Just as poster to SOPW wont pay any attention to comments to a code review request that say things like "your code sucks". Now when that message comes along with a detailed and useful analysis then its a different story.

        Getting back to more constructive thoughts. My current plan wrt to what you want is to provide a way to window the users private scratchpad in a nodelet. Thats an easy proof-of-concept of what you want and can easily be modified to use a $VARS slot instead. It also nicely meshes with existing add to scratchpad functionality so IMO it gets us both soultions for minimal cost. If people really dont see the need for the fancier version once we demonstrate the proof of concept then I will do the work to offline the recent changes.

        Thanks for your comments. They are very useful for future work. I will also begin making pre-announcments of my design intentions and plans so that folks can vent their spleens before I put in all the effort to actually provide what they request.


        ---
        demerphq

          First they ignore you, then they laugh at you, then they fight you, then you win.
          -- Gandhi