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

AppNavigationItem improved editing mode and AppNavigationNewItem component #1277

Merged
merged 45 commits into from
Nov 23, 2020

Conversation

JonathanTreffler
Copy link
Contributor

@JonathanTreffler JonathanTreffler commented Aug 11, 2020

@jancborchardt and I decided to create a new mode for the AppNavigationItem, to add something similar to the editable mode, but without the editing part, just a inline input box that apperas when the logo is clicked and a event when the new input gets acccepted.
We will need this for the News App Vue Rewrite.
I think @jancborchardt can explain this much better than me.

  • implement behaviour
  • make variable names more useful
  • update docs
  • code cleanup
  • move into dedicated component (Work in progress)
  • variable name cleanup
  • Copyright header
  • re-implement automatic focus of the input
  • final testing
  • Screenshot

Screenshots

image

when clicked:
image

@JonathanTreffler JonathanTreffler added enhancement New feature or request 1. to develop Accepted and waiting to be taken care of component Component discussion and/or suggestion feature: app-sidebar Related to the app-sidebar component labels Aug 11, 2020
@JonathanTreffler JonathanTreffler marked this pull request as draft August 11, 2020 11:01
@jancborchardt
Copy link
Contributor

Context: This is for the "New folder" functionality. There the entry should inline switch to become an autofocused input field where you can put in the folder name.

We also need the same for Mail for the action menu where we currently directly show an input field for folder creation, which could be more elegant. cc @ChristophWurst @GretaD

@JonathanTreffler JonathanTreffler changed the title AppNavigationItem Inline Button AppNavigationItem new Item mode Aug 12, 2020
@dartcafe
Copy link
Contributor

I'm just curious.. Do you mean something like this:
https://github.com/nextcloud/polls/blob/master/screenshots/overview.png

@JonathanTreffler
Copy link
Contributor Author

Not really. There are no options like checkboxes. Its not expanding and just one text input

@JonathanTreffler
Copy link
Contributor Author

The behaviour we want works now. I will rename some variables to make the code more understandable, but then its done.

@JonathanTreffler
Copy link
Contributor Author

@jancborchardt Should we force the icon to be a "+" in new Item mode or should the developer choose that ?

@JonathanTreffler
Copy link
Contributor Author

@jancborchardt You said in the Talk group the text should never move (in editing or add item mode). I have implemented that in add item mode and will also implement that in the editing mode.
https://5f3d0ff2e9046d0008918800--nextcloud-vue-components.netlify.app/#!/Components/App%20containers/AppNavigationItem/11

@JonathanTreffler
Copy link
Contributor Author

(editing mode is currently broken here, but i will fix that)

@JonathanTreffler
Copy link
Contributor Author

@jancborchardt You said in the Talk group the text should never move (in editing or add item mode). I have implemented that in add item mode and will also implement that in the editing mode.

done

editing mode is currently broken here, but i will fix that

done

@JonathanTreffler
Copy link
Contributor Author

@jancborchardt Should we force the icon to be a "+" in new Item mode or should the developer choose that ?

@jancborchardt Have you decided yet ?

@JonathanTreffler JonathanTreffler changed the title AppNavigationItem new Item mode AppNavigationItem: "new Item" mode and improved editing mode Aug 29, 2020
@skjnldsv
Copy link
Contributor

@jancborchardt Have you decided yet ?

Hey! Sorry, Jan is in (well deserved) vacations now :(

@skjnldsv skjnldsv added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of component Component discussion and/or suggestion labels Aug 29, 2020
@JonathanTreffler
Copy link
Contributor Author

Changes i made to the editing mode:

(I will probably get rid of the empty space at the end)

@JonathanTreffler
Copy link
Contributor Author

I will probably get rid of the empty space at the end

done

@JonathanTreffler JonathanTreffler force-pushed the AppNavigationItem-inline-button branch 2 times, most recently from 9e012ad to 9181384 Compare September 9, 2020 07:34
Signed-off-by: TessyPowder <tessy.powder@gmail.com>
Signed-off-by: TessyPowder <tessy.powder@gmail.com>
Signed-off-by: TessyPowder <tessy.powder@gmail.com>
Signed-off-by: TessyPowder <tessy.powder@gmail.com>
Signed-off-by: TessyPowder <tessy.powder@gmail.com>
Signed-off-by: TessyPowder <tessy.powder@gmail.com>
Signed-off-by: TessyPowder <tessy.powder@gmail.com>
…m input box

Signed-off-by: TessyPowder <tessy.powder@gmail.com>
Signed-off-by: TessyPowder <tessy.powder@gmail.com>
Signed-off-by: TessyPowder <tessy.powder@gmail.com>
@skjnldsv
Copy link
Contributor

@JonathanTreffler could you rebase and swuash the commits please? 45 is quite a lot!

@skjnldsv
Copy link
Contributor

Let's try to get this in everyone. Or else it'll stay idle for too long

@skjnldsv
Copy link
Contributor

Looks good, docs are 👌
The only thing missing (please do a followup) is being able to open the AppNavigationNewItem with keyboard navigation

@skjnldsv skjnldsv merged commit af54715 into master Nov 23, 2020
@skjnldsv skjnldsv deleted the AppNavigationItem-inline-button branch November 23, 2020 13:51
@skjnldsv
Copy link
Contributor

could you rebase and swuash the commits please? 45 is quite a lot!

Ah, I thought you did already.
Too late 😕

@powerpaul17
Copy link

Sorry if this is the wrong place to post this, but I cannot import the AppNavigationNewItem component. Seems it was forgotten to add it to the build.

@skjnldsv
Copy link
Contributor

@powerpaul17 what components version are you using?

@skjnldsv
Copy link
Contributor

AH, it was forgotten in the exports

@JonathanTreffler
Copy link
Contributor Author

Ah, I thought you did already.

Sorry. I was pretty busy lately.

@powerpaul17
Copy link

@powerpaul17 what components version are you using?

The latest release: 3.3.1

AH, it was forgotten in the exports

That's what I thought. I would have created a fix myself but I'm not familiar with the repo structure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature or request feature: app-sidebar Related to the app-sidebar component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants