Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

[10:45:11 CDT(-0500)] <jgribonvald> i will wait so

[10:46:04 CDT(-0500)] <EricDalquist> I think the change looks good

[10:46:10 CDT(-0500)] <EricDalquist> but there are a lot of whitespace changes

[10:46:13 CDT(-0500)] <EricDalquist> which make it hard to read

[10:46:26 CDT(-0500)] <jgribonvald> yes i can't do things better

[10:46:45 CDT(-0500)] <jgribonvald> my development env delete all useless whitespaces

[10:46:53 CDT(-0500)] <jgribonvald> when i edit a file

[10:46:54 CDT(-0500)] <EricDalquist> ah, you can't turn that off?

[10:47:29 CDT(-0500)] <EricDalquist> I mean I agree with cleaning up whitespace issues

[10:47:38 CDT(-0500)] <EricDalquist> so one thing you could do in the future

[10:47:42 CDT(-0500)] <EricDalquist> is fix the whitespace

[10:47:43 CDT(-0500)] <EricDalquist> commit

[10:47:47 CDT(-0500)] <EricDalquist> then make the actual change

[10:47:59 CDT(-0500)] <EricDalquist> then we have two commits, one with the whitespace changes and one with the content change

[10:48:04 CDT(-0500)] <EricDalquist> that makes merging a lot easier

[10:48:28 CDT(-0500)] <jgribonvald> ok i will wtach on that for futur so

[10:48:47 CDT(-0500)] <jgribonvald> So my pull solve the "bug"

[10:48:56 CDT(-0500)] <jgribonvald> but not the user UI

[10:49:02 CDT(-0500)] <EricDalquist> ok

[10:49:08 CDT(-0500)] <jgribonvald> where some things could be done

[10:49:11 CDT(-0500)] <EricDalquist> yeah

[10:49:17 CDT(-0500)] <EricDalquist> well the UI I'm not as familiar with

[10:49:39 CDT(-0500)] <jgribonvald> my PR push a way to do things like sending message

[10:50:13 CDT(-0500)] <EricDalquist> yeah

[10:50:20 CDT(-0500)] <EricDalquist> so we can get that merged

[10:50:21 CDT(-0500)] <jgribonvald> like "portlet Added" or "can't add portlet" or "can't save layout"

[10:50:41 CDT(-0500)] <jgribonvald> but my problem now

[10:50:51 CDT(-0500)] <jgribonvald> is to show these messages to the user

[10:51:22 CDT(-0500)] <jgribonvald> and/or a way to activate/disable the Add portlet

[10:51:56 CDT(-0500)] <jgribonvald> for current tab that are locked

[10:52:24 CDT(-0500)] <jgribonvald> for messages it's not a big difficult, but what is the best ?

[10:53:44 CDT(-0500)] <jgribonvald> do you know who has made the jquery code for up-layout-gallery.js ?

[10:54:46 CDT(-0500)] <jgribonvald> after to activate/disable the Add portlet it's a bit difficult do to some references

[10:55:03 CDT(-0500)] <jgribonvald> *due to

[10:55:08 CDT(-0500)] <EricDalquist> I'm not sure, it should be in the commit history of the file

[10:55:19 CDT(-0500)] <jgribonvald> ah yes i didn't check that

[10:58:32 CDT(-0500)] <jgribonvald> EricDalquist: else do you see a way to render such messages to users ?

[10:58:54 CDT(-0500)] <EricDalquist> I'm not familiar enough with the UI code to really have an answer therre

[10:59:13 CDT(-0500)] <jgribonvald> authors are Jen Bourey and the main editor Matthew Polizzotti

[10:59:19 CDT(-0500)] <EricDalquist> you could try emailing the dev list an explicitly asking Matt Polizzotti

[10:59:25 CDT(-0500)] <EricDalquist> to see if he can help

[10:59:37 CDT(-0500)] <jgribonvald> ok, i will see so

[10:59:51 CDT(-0500)] <jgribonvald> but i can make changes, just need some advices (tongue)

[11:00:51 CDT(-0500)] <jgribonvald> I can add an element like it's done when a tab doesn't have a content, with the message "You can customize ..."

[11:01:32 CDT(-0500)] <jgribonvald> or open a dialog box during few seconds or with a confirm before coming back

[11:04:32 CDT(-0500)] <jgribonvald> I have a concept, so i needed to talk a bit about (tongue)