Fixed; Issue #10 ( Spells from non-system compendium show up in items tab.)
compendium-browser.js
- loadAndFilterItems(): Refactor test on content to switch on tab (spells, feats, items) and if on item type
- getFilterResult() -> passesFilter()
- Changed Reset Filters prompt to be consistent
- Fully fixed Issue #4 (filters still showing after reset)
compendium-browser.js
- Clear all filters to match displayed
*-browser.html
- Substitute more localization tags
template.html
- Fixed unlisted bug for content (repeated #if showSpellBrowser)
en.json
- Add localization for "Reset Filters" prompt
- Fixed: Error on dragging item from Compendium Browser #9
- Partially Fixed: Issue #4 (filter fields don't get cleared)
-- Is clearing on the current tab but getting out of sync otherwise on other tabs
compendium-browser.js:
- activateItemListListeners(): Remove spurious li.parents (wasn't being used anyway)
- activateListeners(): Add this.refreshList to say what tab we should get data for and then re-render
- Working with Loading and Maximum message at top
- Reset Filters is not resetting filter display
compendium-browser.js
- Change preload to maxLoaded and display a message to filter if you want more
- replaceList(): Pass updateLoading function (with max warning) to renderItemData
en.json
CMPBrowser.SETTING.Preload -> Maxload
CMPBrowser.LOADING.Message, .MaxLoaded: ADDED
*-browser.html
- Add messages line, convert to flexcol
*-browser-list.html
- Use same argument (listItems)
loading.html
- Message (including maximum if found)
compendium-browser.css
- Add .list-area styling consistent with flex
- Adjust max-width
- Remove ul setting of width
- Correct loading/reloading message
compendium-browser.js
- Correct Loading... message on initial tab, but not on tab switch
- replaceList(): Use new loading.html template
loading.html: ADDED
- Add Loading... message for NPCs
- Don't reload if tab already has data, unless you reset filters
compendium-browser.js
- replaceList(): Add options.force to force reload even if there's data (e.g. if you reset filters)
NPC initial load working and subsequent filtering/ordering
compendium-browser.js
- Fix NPCs to use loadAndFilterNpcs
loadAndFilterNpcs(): ADDED
- On settings changed, just do new render()
- decorateCompendiumEntry() -> decorateItem()
npc-browser.html
- Use correct css (cr vs cr-display etc)
- Simplify hidden filtering data fields
module.json
- Version bump
Display static "Loading..." message the first time tab is shown
-> Want it with dynamic results and to not replace existing data
feat-browser.html, spell-browser.html
- Use items array on first load so that we pick up "Loading..." message
Auto-populates the visible tab (based on what is meant to be displayed)
- Needs a Loading... progress indicator in the data area + remember if data has been loaded already
compendium-browser.js
- afterRender(): Hook called after app is rendered to populate initial data
- _onChangeTab(): Override to load data in this tab when you switch tabs
- Move observer to be an instance property
en.json
- Add tab name tags
template.html
- Use tab name tags
- Correct compactItem for feats and items required display items
- Rename itemType -> browserTab to differentiate candidate item's type from the tab it appears on (spell, feat/class, item, NPC)
- Fixed: Was calling the wrong sort for feat and NPC
All tabs now showing values (although missing correct decorations)
compendium-browser.js
- loadAndFilterItems(): test for item5e was skipping items called loot, equipment etc.
Feats seem to be working (missing fields)
Items now showing
NPCs operating erratically
compendium-browser.js
- Add the partials for npc, feat, item and the backing code
*-browser-list.html: ADDED
(partials for changing data)
Generalize the spell list reload and confirm spells still working
compendium-browser.js
- triggerSortSpells() -> triggerSort()
- replaceSpells() -> replaceList()
- renderSpellData() -> renderItemData()
- Remove list/subjects settings and replace with this.replaceList() throughout