Skip to content
This repository has been archived by the owner on Jun 22, 2023. It is now read-only.

[HOLD] List view switcher docs #749

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Blackbaud-ToddRoberts
Copy link
Contributor

No description provided.

@Blackbaud-AdamFunderburk
Copy link
Contributor

The images in the States table do not require borders and do not need to follow 290/600 width requirements.

@Blackbaud-AdamFunderburk
Copy link
Contributor

Is it worth pointing out that when the list view switcher transforms to a dropdown on small viewports, it adopts a unique style of the dropdown that is not standard: icon-plus-caret?

Worth hyperlinking to Dropdown in that paragraph?

@Blackbaud-AdamFunderburk
Copy link
Contributor

We might want to include "Radio button" in the Related information links at the bottom. The same could be true for Dropdown, but I'm not convinced it's appropriate.

I think this is actually the first place inside the system that directly consumes the icon radio buttons, so it can really benefit consumers who see it "for the first time" to be able to get to the Radio button docs.

@blackbaud-johnly blackbaud-johnly changed the title List view switcher docs [HOLD] List view switcher docs Jan 9, 2020
blackbaud-johnly and others added 2 commits February 3, 2020 10:25
* Doc tweaks to list view switcher docs

* ux content edits

* Doc tweaks, skyux install, and resolve conflict.

* Add note with demo

Co-authored-by: Blackbaud-ToddRoberts <todd.roberts@blackbaud.com>
@Blackbaud-BobbyEarl
Copy link

@Blackbaud-BobbyEarl
Copy link

@blackbaud-johnly blackbaud-johnly changed the title [HOLD] List view switcher docs List view switcher docs Feb 3, 2020
@Blackbaud-BobbyEarl
Copy link

@blackbaud-johnly blackbaud-johnly changed the title List view switcher docs [HOLD] List view switcher docs Feb 3, 2020
Copy link
Contributor

@Blackbaud-DeniseCisneros Blackbaud-DeniseCisneros left a comment

Choose a reason for hiding this comment

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

Looks good.

Unless I just overlooked it, the action buttons on the list page need to be updated to add list view switcher, yeah?

@Blackbaud-TrevorBurch
Copy link
Member

Closing as we decided not to document this in favor of data manager

@blackbaud-johnly
Copy link
Contributor

Hey @Blackbaud-TrevorBurch, I asked @Blackbaud-ToddRoberts recently about whether we can just close this PR, and he said the content is still relevant but with the switch to data manager rather than list it would need to live elsewhere. also not sure if it is worth the effort to even put it in the default theme docs or just redo it for modern. I'm not familiar with the content, but it might be worth keeping this PR open as a reminder to circle back to it. I'm going to reopen it for now, but feel free to re-close it if we addressed the relevant docs in data manager.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants