Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added draw of rectangle in 'onEmptyViewClicked' and listener if the same date is clicked twice. (Similar to Google Calendar App). #160

Closed
wants to merge 4 commits into from

Conversation

asolis90
Copy link

Added functionality to draw rectangle in 'onEmptyViewClicked' and listener if the same date is clicked twice. Also added option for it to change the color of the rectangle.
screenshot_2015-07-22-17-03-50

asolis90 added 4 commits July 22, 2015 00:54
…ame date is clicked twice. (Similar to Google Calendar App).
…ame date is clicked twice. (Similar to Google Calendar App).
…ame date is clicked twice. (Similar to Google Calendar App).
…ame date is clicked twice. (Similar to Google Calendar App).
Calendar day = (Calendar) mToday.clone();
day.add(Calendar.HOUR, 6);
int leftDaysWithGaps = (int) -(Math.ceil(mCurrentOrigin.x / (mWidthPerDay + mColumnGap)));
//iterate thru days
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(through)
Please correct english 😄 (feature is cool, though)

@lolobosse
Copy link
Contributor

That's a cool feature but I do not think it has to be activated by default. I would love to have a boolean to activate or deactivate this mode directly in the XML (something like activateDoubleClickListener).

Cool work 🎉 👍 ❗

@asolis90
Copy link
Author

Thanks! 😄 I worked on it a couple days ago, wanted to share it with you guys.
When you mean activate by boolean you mean the drawing of the rectangle and the doubleClickListener?

@lolobosse
Copy link
Contributor

Yes that your code only works if I activate the feature (so you basically parse the boolean with a false as a default behaviour and then, you activate your doubleClickListener only if there this boolean is true).

@entropitor
Copy link
Contributor

Indeed, this should be an optional feature.

@wwahmed
Copy link

wwahmed commented Feb 15, 2016

Any chance of this getting updated and merged -- it's a nice optional feature, but looks like it was developed on a much older revision and some updates might be necessary?

@asolis90
Copy link
Author

Hey guys sorry about that been busy lately.. Will try to work on this feature this week..

@entropitor
Copy link
Contributor

@asolis90 @jhoobergs is also working on this. He has a recent version.

@jhoobergs
Copy link
Contributor

@asolis90 Here is my fork: https://github.com/Quivr/Android-Week-View/tree/Add_events
I think it is almost complete, but the text inside the rectangle isn't calculated properly

@asolis90
Copy link
Author

asolis90 commented May 6, 2016

hey guys, don't think i forgot about this feature.. lol Today I spent some time in it.. Its almost done and fully functional 😄

@entropitor
Copy link
Contributor

@asolis90 @jhoobergs already has a working version. (One that's even in production already).

@wwahmed
Copy link

wwahmed commented May 11, 2016

But is @jhoobergs planning to submit a pull request for it?

On Mon, May 9, 2016 at 3:17 AM, Jens Claes [email protected] wrote:

@asolis90 https://github.com/asolis90 @jhoobergs
https://github.com/jhoobergs already has a working version. (One that's
even in production already).


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#160 (comment)

@asolis90
Copy link
Author

I have tested his version and it is not the desired way to go about doing this. The empty event is conflicting with other events when its drawn, also the empty event rectangle should be drawn hourly and is not being handled correctly. I'll close this PR since its off track and open a new one. Please review my changes and if possible give some feedback.

@asolis90 asolis90 closed this May 12, 2016
@asolis90
Copy link
Author

ref #321

kimkyuhwan pushed a commit to AE-TEM/Android-Week-View that referenced this pull request Nov 20, 2021
* fix for concurrency bug

* minimized diff

* used CopyOnWriteArrayList

Co-authored-by: Felix Felten <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants